Activity CacheAdminA_trunk2010/WSimpl/PM/Maatwerk/MaatwerkService/DotNet

12:02 pm

Wim Vermeulen

More
CR-2486
started review

[ICT-3602][rvPVR] Misleidende %objlasterror in HalffabrikatenFromJsonNaarDomConverter opkuisen

5:25 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Er bestaan nog extra testen voor convertoren, je moet es kijken naar mijn commit 68434, daar staan bijna alle in behalve 'appsnaarecon', das de enige wss

12:42 pm

Ah oke, op deze manier heb je het opgelost haha, al de comments voor niets http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif. Ik heb daar nog altijd mijn...

Ah oke, op deze manier heb je het opgelost haha, al de comments voor niets . Ik heb daar nog altijd mijn bedenkingen bij om het onder die Data.bomkenmerken te brengen, we verliezen zo toch wel een simpel overzicht over alles zoals blijkt uit mijn opmerkingen, maar dat is mijn mening

12:37 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Als stuklijst hier niet moet verwerkt worden moet die wel worden toegevoegd aan bovenstaande Converteer method. Anders zal er helemaal niets gebeuren voor deze property

12:36 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Stuklijst heb je gestoken onder Data.bomkenmerken en niet data.kenmerken. Deze method gaat code genereren, je moet maar eens kijken in de gecompileerde versie van deze klasse. Daar ga je op regel 1...

Stuklijst heb je gestoken onder Data.bomkenmerken en niet data.kenmerken. Deze method gaat code genereren, je moet maar eens kijken in de gecompileerde versie van deze klasse. Daar ga je op regel 182 zien wat ik bedoel

12:29 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Ik heb me hier al vragen over gesteld of het bij de mat (en mss ook de rest als ik er nu over nadenk) ook niet verkeerd zit. Bij LBX heb je daar die Clone ervan en ga je niet de oorspronkelijke ken...

Ik heb me hier al vragen over gesteld of het bij de mat (en mss ook de rest als ik er nu over nadenk) ook niet verkeerd zit. Bij LBX heb je daar die Clone ervan en ga je niet de oorspronkelijke kenmerken overschrijven. Hier werk je zonder clone en ga je eigenlijk het oorspronkelijke object verder aanvullen. Dit is niet echt gewenst gedrag aangezien we dan ook mogelijks een andere canonische waarde uitkomen verder in de flow. Dat zit allemaal jammer genoeg heel tricky in elkaar :|. We willen eigenlijk gewoon het object een beetje manipuleren vooraleer we het sturen naar .net maar we willen die aangepaste versie niet opslaan in de DB. Veiliger zou zijn om eigenlijk een convertor te hebben in .net maar dat is net het probleem dat we hier oplossen

12:25 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

TBX kenmerken

12:25 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Hier ook nog mvx

12:24 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Ik zou hier toch een soort van if else opzetten want voor TBX ben ik hier ook onwetend tegen de muur gelopen omdat het altijd slaagde. Stel dat hier later een variant op komt dat zal het mss ten or...

Ik zou hier toch een soort van if else opzetten want voor TBX ben ik hier ook onwetend tegen de muur gelopen omdat het altijd slaagde. Stel dat hier later een variant op komt dat zal het mss ten orechte blijven werken allemaal en zit je later met mogelijke miserie

12:23 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

En hier ook maar het was wss gewoon kwestie van een korttekstbepaler te hebben http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/emoticons/tongue.gif

En hier ook maar het was wss gewoon kwestie van een korttekstbepaler te hebben

12:22 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Hier staat nog een MVX base

12:22 pm

Peter Vercoutter

More
Re: CR-2472
commented on review

Is adapter de enige nodige rol? Willen we niet per halffabrikaat er een rol aan toekennen? Geen idee hé, maar een bedenking http://subversion02.vanhoecke.be/static/ogdo0b/2static/images/wiki/icons/...

Is adapter de enige nodige rol? Willen we niet per halffabrikaat er een rol aan toekennen? Geen idee hé, maar een bedenking

11:56 am

Jelle Baele

11:50 am

Jelle Baele

More
Re: CR-2472
commented on review

Deze klasse hoort niet bij dit kaartje. Ik had dit gecommit met de verkeerde message en had mijn commit message aangepast, maar is precies niet opgepikt door crucible.

11:48 am