Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-4991] Implementatie voor nieuwe KlantInstelling KartonnenPallet

- Reviewopmerking: Kartonnen pallet heeft geen kleine versie, dus op overzicht batch mag gewoon 'Kartonnen pallet' staan

[ICT-5051][rvNVT] FOP: DocBase Upgrade - Clean up caché-code na de go-live

- Logging toegevoegd aan method PrintKlantEtiket(), die de routine PRINT^HADETIK oproept.

[ICT-5051][rvNVT] FOP: DocBase Upgrade - Clean up caché-code na de go-live

- Verwijzingen naar FOP (eerder van Print01) aangepast : nu \\PRINT01-PRD\ICT6050 i.p.v. \\PRINT01\ICT

- Het blijft vreemd dat PRINT^HADETIK verwijst naar een laserprinter nl. "|PRN|\\FOP\ICT" om etiketten af te printen (via bartender). Het is ook nog niet geweten of hier echt een bestaande printer moet aangesproken worden, of misschien mag het ook een dummy string zijn. Testen vanuit het actieve-batches scherm is lastig want de aansturingActie "Print KlantEtiketten" kan slechts 1x uitgevoerd worden. Dus herhaaldelijk testen (lees: trial-and-error) is moeilijk.

[ICT-4476] [rvJBA] VHLOG MES M1 - Klantlabel voor only's door Caché laten afdrukken

- Verschil in enum waarde!!

[ICT-5051][rvNVT] FOP: DocBase Upgrade - Clean up caché-code na de go-live

- Verwijzingen naar FOP aanpassen : Wel vreemd dat PRINT^HADETIK verwijst naar een laserprinter nl. "|PRN|\\FOP\ICT"; aangepast naar \\PRINT01\ICT (LET OP : deze share bestaat wel op beide servers)

spatie voor de K? Hebben we ook kleine kartonnen palletten? Ik vraag het maar omdat ..BepaalGroottePallet "klein" en "groot" kan teruggeven, maar ik heb nog nooit een kleine kartonnen pallet gezie...

spatie voor de K?

Hebben we ook kleine kartonnen palletten? Ik vraag het maar omdat ..BepaalGroottePallet "klein" en "groot" kan teruggeven, maar ik heb nog nooit een kleine kartonnen pallet gezien hier. Ik kan hier natuurlijk ook compleet verkeerd zijn.

[ICT-4991] Implementatie voor nieuwe KlantInstelling KartonnenPallet
[ICT-4991] Implementatie voor nieuwe KlantInstelling KartonnenPallet
[ICT-4932] [XS] [rvPVR] Vertalen VhConfig binnen Cache

- Kenmerk HasMat is veranderd naar HasCustomMat

  1. … 1 more file in changeset.
[ICT-5204] [rvJBA] Encoway SpaceStep voorzien in cache
[ICT-5204] [rvJBA] Encoway SpaceStep voorzien in cache
[ICT-5204] [rvJBA] Encoway SpaceStep voorzien in cache

- Afbeelding van SST en LBX werd opnieuw berekend, mag niet voor encowayproducten gezien een ander systeem deze al heeft berekend

[ICT-5184] [rvJBA] Verplaatsen van batch in de pitching excel
[ICT-5184] [rvJBA] Verplaatsen van batch in de pitching excel
[ICT-5184] [rvJBA] Verplaatsen van batch in de pitching excel

- backgroundjob aanpassen zodat er niet te veel tijd gaat over het inlezen en verwerken van die data

    • -2
    • +28
    ./Planning/ResourcePitchingService.cls.xml
[ICT-5184] [rvJBA] Verplaatsen van batch in de pitching excel

- Nieuwe aansturingactie toevoegen zodat Planning zelf data uit de tabel kan verwijderen

  1. … 7 more files in changeset.
[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode

- reviewopmerkingen

Vrij veel code in 1 method, ik zou die wat meer opsplitsen http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif. Ik zie ook geen UT's, dat kan op die manier ...

Vrij veel code in 1 method, ik zou die wat meer opsplitsen . Ik zie ook geen UT's, dat kan op die manier wel opgelost worden

Commentaar mag weg

Commentaar mag weg

Is deze implementatie niet heel gelijkaardig aan die voor MVX? Lijkt mij dat deze in de common klasse kan http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif

Is deze implementatie niet heel gelijkaardig aan die voor MVX? Lijkt mij dat deze in de common klasse kan

[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode
[PRJ051-26] [rvJBA] Papier voor aansturing BWC freesmachine (Homag) vervangen door barcode

- Correctie na fysieke testen

  1. … 9 more files in changeset.
[PRJ051-34] Nieuwe rollen voor losse onderdelen (schroeven, scharnieren) bij SPT

- Reviewopmerkingen

LosOnderdeelAantal in plaat van 'Nummer' lijkt mij iets logischer

LosOnderdeelAantal in plaat van 'Nummer' lijkt mij iets logischer

string '$X$' kan je in een aparte variabele plaatsen, zodat je geen extra commentaar nodig hebt

string '$X$' kan je in een aparte variabele plaatsen, zodat je geen extra commentaar nodig hebt

BSKey zou ik hernoemen naar 'BouwsteenKey' voor de duidelijkheid

BSKey zou ik hernoemen naar 'BouwsteenKey' voor de duidelijkheid