Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
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
ProjectSettings eruit gehaald

ProjectSettings eruit gehaald

Done!

Done!

$$$True ipv 1

$$$True ipv 1

geen .%New() in de methods. Afzonderen naar de constructor (zelfs als ge niet injecteert)

geen .%New() in de methods. Afzonderen naar de constructor (zelfs als ge niet injecteert)

Het logogedeelte is niet relevant in deze test. Kunt dat beter wegsmijten om op die manier ietske rapper aan de lezer duidelijk te maken wat de essentie is van deze test. (code wordt maar 1x geschr...

Het logogedeelte is niet relevant in deze test. Kunt dat beter wegsmijten om op die manier ietske rapper aan de lezer duidelijk te maken wat de essentie is van deze test. (code wordt maar 1x geschreven, maar veel keer gelezen)

[VRB488] HXPlanning: Werkposten aansturen: TAX printen op andere A4-printer (ook VRB36):
[VRB488] HXPlanning: Werkposten aansturen: TAX printen op andere A4-printer (ook VRB36):
  • More
  • CR-167
  • finished reviewing
:thumbs up:

:thumbs up:

[VRB419] Geen VH-Logo op onderdelenetiket Du pont.
[VRB419] Geen VH-Logo op onderdelenetiket Du pont.
  • More
  • CR-71
  • finished reviewing
  • More
  • CR-71
  • started review
[BUS582] Du-Pont: eigen logo op losse onderdelendoos-etiketten
[BUS582] Du-Pont: eigen logo op losse onderdelendoos-etiketten