CacheAdminA_trunk2010/BL/PPS/common/impl

OnderdelendoosLogoBepaler.cls.xml

Checkout Tools

Line History

Line Count Graph
Line Count Graph

Stats

Commits this week: 0
Total Committers: 3
Last Commit: 15 Jan 21
Commits this week: 0
Total Lines of Code (LoC): 29
Net change in LoC this week: 0

Commit Activity

Commits By Day In Week
52 week commits volume
Commits By Day In Week
Commits by day
Commits By Day In Week
Commits by hour
Commits By Hour In Day
Commit calendar
 

Most active committers (90 days)

loading
loading
  • last updated a few seconds ago
All Commits Reviews
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Tuesday 02 Aug 2022
10:09 am

Wim Vermeulen

Wednesday 20 Jan 2021
12:37 pm

Wim Vermeulen

Friday 15 Jan 2021
8:16 pm

SamV

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

-Review opmerkingen verwerkt

-$Get("global") =1 gebruikt ipv $data()

-Extra testen toegevoegd voor klant Van Hoecke en een klant die deze global niet heeft

Options
    • -1
    • +1
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 2 more files in changeset.
8:08 pm

Sam Van Hoey

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.

6:55 pm

Wim Vermeulen

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.

6:48 pm

Wim Vermeulen

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 )
6:42 pm

Wim Vermeulen

$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.

6:34 pm

Wim Vermeulen

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?

2:42 pm

SamV

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

-Logo naar logo met kleine letter zo dat de integratietesten niet meer falen, die rekenen op logo met kleine letter

Options
    • -1
    • +1
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 1 more file in changeset.
12:30 pm

SamV

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

-OnderdelenDoosLogoBepaler terug de vorige parameters laten implementer zodat de wijzigingen aan docsv1 niet moeten mee gecommit worden om een mergeconflict met JB te voorkomen

Options
    • -1
    • +3
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 3 more files in changeset.
12:10 pm

SamV

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

-OnderdelenDoosLogoBepaler aangepast zodat deze nu ook de global gebruikt zoals de labellogobepaler

-todo: veralgemenen zodat er maar 1 bepaler nodig is, mogelijks in Klant.GeefLogo(parameter)

Options
    • -12
    • +13
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 2 more files in changeset.
Friday 16 Oct 2020
8:53 am

Wim Vermeulen

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.

Friday 09 Oct 2020
10:31 am

Sam Van Hoey

10:31 am

Sam Van Hoey

[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck
[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck
Thursday 24 Sep 2020
12:26 pm

SamV

[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck

-KlantNummer in een aparte variabele gezet

Options
    • -2
    • +5
    ./OnderdelendoosLogoBepaler.cls.xml
12:19 pm

SamV

[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck

-code aanpassing door gebruik van andere global

Options
    • -2
    • +2
    ./OnderdelendoosLogoBepaler.cls.xml
11:42 am

SamV

[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck

-Logo minder hardcoded er in gezet, kijkt nu naar de global zoals alle andere locaties

-Moet misschien ook nog gerefactored worden maar moet na gevraagd worden of dit voor de andere multipersonalities ook mag, dan kan de LabelLogoBepaler gebruikt worden

Options
    • -6
    • +8
    ./OnderdelendoosLogoBepaler.cls.xml
Wednesday 16 Sep 2020
3:26 pm

SamV

[ICT-1797] Wijzigen etiketten bestellingen MP Van Raemdonck

-Logo van Van Raemdonck toegevoegd aan OnderdelendoosLogoBepaler.cls.xml

Options
    • -0
    • +1
    ./OnderdelendoosLogoBepaler.cls.xml
Monday 28 Oct 2019
3:21 pm

Jo Willems

[ICT-986] MP HaefeleDE opzetten

-initiele commit.

Options
    • -0
    • +1
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 107 more files in changeset.
Friday 21 Apr 2017
11:38 am

Kurt Callebaut

Thursday 20 Apr 2017
4:32 pm

Jo Claes

  • More
  • CR-167
  • finished reviewing
4:31 pm

Jo Claes

:thumbs up:

:thumbs up:

Wednesday 05 Apr 2017
2:29 pm

Kurt Callebaut

[VRB419] Geen VH-Logo op onderdelenetiket Du pont.
[VRB419] Geen VH-Logo op onderdelenetiket Du pont.
12:05 pm

Admin

[VRB419] Geen VH-Logo op onderdelenetiket Du pont.

- Test gewijzigd ivm logobepaler -> mocken.

Options
    • -1
    • +1
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 3 more files in changeset.
11:51 am

Admin

[VRB419] Geen VH-Logo op onderdelenetiket Du pont.

- Klant enu gebruikt.

Options
    • -3
    • +3
    ./OnderdelendoosLogoBepaler.cls.xml
11:00 am

Admin

[VRB419] Geen VH-Logo op onderdelenetiket Du pont.

- Test nog te refactoren ivm logo. Werking testen bij TVE (cache05 nodig)

Options
    • -0
    • +19
    ./OnderdelendoosLogoBepaler.cls.xml
  2. … 3 more files in changeset.