Checkout Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
[ICT-1082] [rvTVE] EC: VH4: MP: Hahle multipersonality (Finland)
[ICT-1082] [rvTVE] EC: VH4: MP: Hahle multipersonality (Finland)
Moet blijkbaar ook niet wegens deprecated.

Moet blijkbaar ook niet wegens deprecated.

op de zelfde plaats als al de andere mp's http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif (nergens dus alleen VH wordt getest.)

op de zelfde plaats als al de andere mp's (nergens dus alleen VH wordt getest.)

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

Logica is gewoon omgedraaid. nieuwe personalities zullen geen code wijzigen meer nodig hebben.

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

JoC was het hiet niet mee eens. als je de enum gebruikt zie je de impact niet als deze verandert

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

De ontvanger lijkt mij een goede kandidaat om toe te voegen aan de klantinstellingenService

Opkuis van deze klasse is wenselijk.

Opkuis van deze klasse is wenselijk.

Probleempje met de TAB toets? http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

Probleempje met de TAB toets?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Is dit in afwachting van een echte Noorse vertaling? Of moet op een ander niveau een fallback voorzien worden naar het Engels?

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Waarom zijn enkele personalities uitgeschakeld? --> inschakelen of wegsmijten (boyscout)

Is deze test overbodig?

Is deze test overbodig?

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Copy/paste code: gemeenschappelijke code groeperen en parametriseren

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Leesbaarheid verhogen door gebruik te maken van de personality enumeratie.

Waar wordt dit getest?

Waar wordt dit getest?

Unishop is uitgefaseerd --> renamen

Unishop is uitgefaseerd --> renamen

BeslagTechnikk ontbreekt

BeslagTechnikk ontbreekt

Veel duplicate code, ik maak een aparte issue aan

Veel duplicate code, ik maak een aparte issue aan

[ICT-774] EC: MP: BeslagTeknikk multipersonality opzetten
[ICT-774] EC: MP: BeslagTeknikk multipersonality opzetten
Ook bestaande code natuurlijk. Bij refactor es zien om personality enums te gebruiken. .

Ook bestaande code natuurlijk. Bij refactor es zien om personality enums te gebruiken.
.

Misschien moeten we ook es zien om de winkelkar te voorzien van een accept method zoals bij "APPS.EC.Winkelkar.dto.Winkelkar" zodat we visitors kunnen gebruiken om maatwerklijnen aan te passen.

Misschien moeten we ook es zien om de winkelkar te voorzien van een accept method zoals bij "APPS.EC.Winkelkar.dto.Winkelkar" zodat we visitors kunnen gebruiken om maatwerklijnen aan te passen.

Aangezien de dto in memory is, zou ik de waarde steeds op "true" zetten, zonder een extra check uit te voeren. (de uitkomst is in alle gevallen "true") De "if"'s hoeven niet, tenzij we via logging ...

Aangezien de dto in memory is, zou ik de waarde steeds op "true" zetten, zonder een extra check uit te voeren. (de uitkomst is in alle gevallen "true")
De "if"'s hoeven niet, tenzij we via logging willen weten welke waarde wel of niet aangepast is ?

oude code natuurlijk, maar een if en een conditionele quit kunnen één worden. desnoods in private method.

oude code natuurlijk, maar een if en een conditionele quit kunnen één worden. desnoods in private method.

oude code natuurlijk, maar een if en een conditionele quit kunnen één worden. http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/smile.gif desnoods in private method.

oude code natuurlijk, maar een if en een conditionele quit kunnen één worden. desnoods in private method.

Misschien in de naam verduidelijken wat de test doet. "factory geeft winkelkaraanpasser van hein weer" of zoiets.

Misschien in de naam verduidelijken wat de test doet. "factory geeft winkelkaraanpasser van hein weer" of zoiets.