DCPRINT2.cls.xml

Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-5186][rvNVT] FOP : aanpassing layout leverbon, op blanco papier i.p.v. logo-papier
[ICT-5186][rvNVT] FOP : aanpassing layout leverbon, op blanco papier i.p.v. logo-papier
[ICT-3691] Blum Polen: eigen leverbon hoofding
[ICT-3691] Blum Polen: eigen leverbon hoofding
Ik heb de method opgekuist, want deze lijkt voorlopig niet meer te zullen terugkomen.

Ik heb de method opgekuist, want deze lijkt voorlopig niet meer te zullen terugkomen.

ik vind iets als dit altijd dubbel: de method kan (functioneel gezien) gewoon weg en de enige reden dat ie blijft bestaan is voor de comments/historiek te kunnen zien. In principe dient daar subver...

ik vind iets als dit altijd dubbel: de method kan (functioneel gezien) gewoon weg en de enige reden dat ie blijft bestaan is voor de comments/historiek te kunnen zien. In principe dient daar subversion/confluence voor, niet de codebase.
Ik heb dat vroeger ook meestal zo gedaan, maar hier (VH) heb ik (van JoC) altijd gehoord: als het niets meer doet, dan wegsmijten. Ik kan daar wel in volgen hoor. Zeker als het wat deftig in een kaartje/confluence uitgelegd staat.

[ICT-2582][rvTVE] MAG: Leverbon: LEVBON_PS nieuwe printerlade voor blanco papier (rechtstreeks transport...
[ICT-2582][rvTVE] MAG: Leverbon: LEVBON_PS nieuwe printerlade voor blanco papier (rechtstreeks transport...
Opmerking in Jira-kaartje: Tom Vermeulen : Zorg je wel dat je een "backup" hebt van het verzendwijze-gedeelte van de global, dus ^RES("KLANT","PI","VERZENDWIJZE") van Cache01, zodat je bij twijfel...

Opmerking in Jira-kaartje:

Tom Vermeulen : Zorg je wel dat je een "backup" hebt van het verzendwijze-gedeelte van de global, dus ^RES("KLANT","PI","VERZENDWIJZE") van Cache01, zodat je bij twijfel kan terugzien naar welke printer een verzendwijze voorheen stond.

Suggestie Quick-and-dirty :
Merge ^Log.TVE.TempRES($H,"KLANT","PI","VERZENDWIJZE") = ^RES("KLANT","PI","VERZENDWIJZE")

Heb je toevallig ook hieraan gedacht? Heeft de nieuwe printer een Stapling-mogelijkheid? Toegegeven : Levbon_DI staat hier ook niet letterlijk bij naam in vermeld, die zit bij de pattern-match "Els...

Heb je toevallig ook hieraan gedacht?
Heeft de nieuwe printer een Stapling-mogelijkheid?
Toegegeven : Levbon_DI staat hier ook niet letterlijk bij naam in vermeld, die zit bij de pattern-match
"Else If (PrinterName?1(1"LEVBON_PS",1"LEVBON".E)) Do"

Indien je weet dat er geen nietjes-console voorzien is, kan je LEVBON_EXPORT ook toevoegen bij de If levbon_Terugname

[ICT-2139] [rvWV] MAG: Leverbon: LEVBON_EXPORT doen gebruiken voor leverbon+pakbon bij klanten in...
[ICT-2139] [rvWV] MAG: Leverbon: LEVBON_EXPORT doen gebruiken voor leverbon+pakbon bij klanten in...
Zet hier eens een commentaarlijn bij in de code : "Review opmerking van WimV : aan de volgende die dit uitbreidt, gelieve dit te refactoren aub, want deze constructie stinkt een beetje!"

Zet hier eens een commentaarlijn bij in de code :
"Review opmerking van WimV : aan de volgende die dit uitbreidt, gelieve dit te refactoren aub, want deze constructie stinkt een beetje!"

ICT-2095 Wijzigen etiketten bestellingen MP Tonny Campe
ICT-2095 Wijzigen etiketten bestellingen MP Tonny Campe
Ja dat was iets van Jo, maar is nu ook rood, is aangepast in revision 63265 door Jo.

Ja dat was iets van Jo, maar is nu ook rood, is aangepast in revision 63265 door Jo.

TIP : Misschien nog iets meer uitleg in de commit message : waarom tests in commentaar gezet? Is dit tijdelijk? Moeten de code eerst terug gefixed worden? Moeten ze later alsnog aangepast worden? M...

TIP :
Misschien nog iets meer uitleg in de commit message : waarom tests in commentaar gezet? Is dit tijdelijk? Moeten de code eerst terug gefixed worden? Moeten ze later alsnog aangepast worden? Mogen ze weg, maar twijfel je nog?
Feel free om dit soort zaken in de commit message toe te voegen, want nu lees ik enkel dat er iets veranderd is in de LogoBepaler.

Nog 2 testmethods toevoegen : *voor klantnummer 4682 *voor klantummer "UT_klantummerNietinIPCom" --> result is logoVH (alé, dat verwacht ik toch http://subversion02.vanhoecke.be/static/ogdo0b/2st...

Nog 2 testmethods toevoegen :

  • voor klantnummer 4682
  • voor klantummer "UT_klantummerNietinIPCom" --> result is logoVH (alé, dat verwacht ik toch )
$D() of $Data() geeft aan of de subnode aanwezig is in de global, ongeacht de waarde die er ingevuld is. Dus het lijkt me veiliger om gebruik te maken van : $Get(...) = 1 want stel dat iemand ooit ...

$D() of $Data() geeft aan of de subnode aanwezig is in de global, ongeacht de waarde die er ingevuld is.
Dus het lijkt me veiliger om gebruik te maken van : $Get(...) = 1
want stel dat iemand ooit de waarde in de global wijzigt van 1 naar 0 , dan zal $Data() nog steeds true zijn.

Eigenaardige if-clause hierboven : Klant is NIET haefeleBE en toch projectSetting voor HaefeleBEnaarEindklant checken. is dat niet heel raar?

Eigenaardige if-clause hierboven :
Klant is NIET haefeleBE en toch projectSetting voor HaefeleBEnaarEindklant checken.
is dat niet heel raar?

Bij klant OptimaPack zie ik hetvolgende : *vorige versie geeft pxLabelKlant.Logo = "" *jou nieuwe versie (via LabelLogoBepaler + Data in ^IPCom) pxLabelKlant.Logo = "logo\Logo Optima 6279.svg" ...

Bij klant OptimaPack zie ik hetvolgende :

  • vorige versie geeft pxLabelKlant.Logo = ""
  • jou nieuwe versie (via LabelLogoBepaler + Data in ^IPCom) pxLabelKlant.Logo = "logo\Logo Optima 6279.svg"


Dat lijkt me niet hetzelfde eindresultaat te zullen geven. Misschien is dat oké, maar het is alvast mijn taak als kritische reviewer om dit op te merken.

[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck
[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck