Crucible - Comment Search

Accordion

Defect Metrics

Statistics on the results

Ranking

Defect Report

Classification

Defect Report

1077 Defects:
435 major defects, and
642 other defects
from 4201 comments

Search Criteria

Search for comments

Review	Name	Created	Custom Fields	Content
CR-33		13 Jan 2017		super
CR-33		13 Jan 2017	Ranking: Minor Classification: Not conforming to standards	ontbrekende enu
CR-25		13 Jan 2017	Ranking: Major Classification: Factually incorrect	Het is de bedoeling dat we per method kunnen switchen. Dus het config item moet op zijn minst ook een vermelding hebben van de method "_Implementatie" is een gebruikt gegeven in de config Items WS.Vhisie4.Winkelkar.WinkelkarService:[MethodNaam]_Implementatie : dus een concreet zal het zijn : WS.Vhisie4.Winkelkar.WinkelkarService:BewaarMaatwerkLijn_Implementatie
CR-25		13 Jan 2017	Classification: Factually incorrect Ranking: Major	Deze deploy klasse mag gedeletet worden en het reeds gemaakte configItem mag verwijderd worden
CR-22		13 Jan 2017		Lijkt mij beter om de ExterneId mee te geven aan de Builder want anders hebben we een deel van de logica hier zitten en een ander deel in de builder
CR-21		16 Jan 2017		Er is gekozen voor deze benaming. Hadden ze op voorhand maar moeten bedenken.
CR-21		16 Jan 2017		Een goeie week geleden heb je gezegd dat we niet offerte id en winkelkar id mogen gebruiken in foutmeldingen omdat het een en dezelfde functionaliteit is. We hebben dus voor externeid gekozen. Dus laat externeid maar staan. WinkelkarId is gewoon de testwaarde. Niet blokkerend.
CR-21		16 Jan 2017		OKay.
CR-21		16 Jan 2017		Laat maar staan, dan is het duidelijk dat er op 2 waarden gecontroleerd wordt.
CR-21		16 Jan 2017		Hieronder staat GeefViaID ?
CR-21		16 Jan 2017		Hierboven staat GeefViaId ? Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories
CR-21		16 Jan 2017		Methods ....ViaExterneId refactored naar ViaID voor Vhisie4.WInkelkar.... Repositories
CR-21		16 Jan 2017		Dan compileert de klasse niet.
CR-21		16 Jan 2017		Dan compileert de klasse niet.
CR-21		16 Jan 2017		Dan compileert de klasse niet.
CR-34		16 Jan 2017	Ranking: Major Classification: Factually incorrect	moeten we er nog woorden aan vuil maken Intern <> Extern
CR-21		16 Jan 2017		thx for the feedback.
CR-34		16 Jan 2017	Classification: Factually incorrect Ranking: Minor	BIDI en BUDI zijn verschillende personen dus lijkt het mij incorrect dat het dezelfde persoon is 'abcde'
CR-34		16 Jan 2017	Ranking: Minor Classification: Inconsistent	Waarom een FakeBIDI en geen FakeBUDI
CR-23		16 Jan 2017		deze test is intussen gewijzigd.
CR-25		17 Jan 2017		Lijkt met een risico om niet de volledige interface te switchen. Bij het apart switchen van klassen bestaat het gevaar dat dummy en productie door elkaar gebruikt wordt waarbij onterecht errors kunnen optreden.
CR-35		17 Jan 2017	Ranking: Minor Classification: Not conforming to standards	Gaan we hier nu al %Newen?
CR-35		18 Jan 2017		Ja, want het is een builder en deze builder gebruikt builder subklassen zie al de andere builders in de DOM.PM.Maatwerk.Calc.HF....
CR-38		18 Jan 2017		Zit een beetje met een dilemma : kwestie van de nakomelingen tegen hun eigen te beschermen of moeten we vertrouwen hebben in de nakomelingen dat die de TDD-technieken in zich gaan hebben daarom ik eerder zou opteren voor volgende constructie If (TBXProduct.UitsparingDataAantalUitsparingen = 1) { } elseif (TBXProduct.UitsparingDataAantalUitsparingen = 2) { } else { BoemPatat("meer dan 2 uitsparingen niet ondersteund") }
CR-38		18 Jan 2017	Ranking: Minor Classification: Not conforming to standards	ofwel dimmen we iets As ofwel als het niet nodig is dan zetten we een set .. want nu zal er geen auto-completion zijn.
CR-36		18 Jan 2017		#dim IngegevenKenmerken As DOM.PM.Maatwerk.Calc.Common.impl.TAORKenmerken = Product.GeefIngegevenKenmerken() If ..IngegevenKenmerkenTypeAPI.IsEenLosseComponentenVerpakking(IngegevenKenmerken) { zou eigenlijk een onliner kunnen worden If ..IngegevenKenmerkenTypeAPI.IsEenLosseComponentenVerpakking(Product.GeefIngegevenKenmerken())
CR-39		18 Jan 2017		Vind het wel raar dat er geen enkele conversie test ( klasse ) is aangepast terwijl er toch overal nieuwe kenmerken gedefinieerd worden
CR-38		18 Jan 2017		I know, I know, staat al een tijdje zo... zal het oplossen in kaartje 2086, want anders zitten we weer met mergebazaar
CR-39		19 Jan 2017		Als op alle laagskes dezelfde schuif op dezelfde manier wordt aangepast (opvullijsten dus), en de conversies waren al snugger genoeg om er mee om te gaan, zullen de conversies(/tests) niet aangeepast moeten worden.
CR-48		25 Jan 2017		Test benaming moet toch eerder zijn als kredietlimiet 1000 is dan geven we 1 door
CR-45		25 Jan 2017		Geen drietaps if elseif else nodig met in den else een exceptie ??
CR-45		25 Jan 2017	Ranking: Minor Classification: Not conforming to standards	Beter een if elseif structuur zodoende niet alle if's moeten gevalideerd worden.
CR-44		25 Jan 2017	Ranking: Major Classification: Factually incorrect	Zou het kunnen dat we hier geen testje voor hebben ?? aangezien er toch ietske is dat waarschijnlijk niet klopt
CR-44		25 Jan 2017		geen .impl. in deze package structuur de impl staat vooraan .. het zijn allemaal impl
CR-44		25 Jan 2017		privatiseren
CR-44		25 Jan 2017		wie is vol ??
CR-44		25 Jan 2017		Dimmen niet nodig bij %New() .. noodzakelijke info ( constructor parameters ) valt van het scherm
CR-44		25 Jan 2017	Ranking: Major Classification: Risk-prone	Waar is het bewijs dat hij niet opgeroepen wordt ? Ik zou in de method op zijn minst één of andere exact aantal keer 0 gezet hebben.. Want door niets te verwachten en niet te verifiëren wordt het niet getest
CR-44		25 Jan 2017	Ranking: Major Classification: Factually incorrect	Waarom wordt de 654321 niet geconverteerd ???
CR-45		26 Jan 2017		Caché is toch snel genoeg? (soit, ben al content dat "PeuterLadeCodeOpen" het gehaald heeft )
CR-48		26 Jan 2017		Het is kort voor "Als de kredietlimiet 1 is (zo zichtbaar in Admin), dan moet er geen 1000 naar AX worden gestuurd, in andere gevallen vermenigvuldigen we wel met 1000 (en zetten wer er 20 nullen achter de komma bij)", maar dat vond ik te lang als omschrijving. (en caché vond dat ook)
CR-45		26 Jan 2017		In principe zouden er geen andere waarden in Product.FrontOndersteuningAantal mogen zitten dan "", "1", "2" of "3" ... en in ItemFS enkel 1,2,3,HS of A. Normaagezien zou er geen exceptie geraised mogen worden... De andere "convert*" methods raisen ook geen excepties. Qua strutctuur: op deze manier is het voor de geïnteresseerde lezer onmiddellijk duidelijk dat bij aantal="" er naar het type gekeken gaat worden. Bon, 't is nen uitleg gelijk een ander.
CR-49		27 Jan 2017		Deze mag eigenlijk EconConfiguratie noemen - dat het toch een correctere naam is
CR-49		27 Jan 2017		Waarom zijn dat opeens DOM kenmerken geworden ?? eigenlijk zou het resultaat een maatwerklijn moeten zijn.
CR-49		27 Jan 2017		Misschien een testje bijbouwen waarin we expliciet zeggen dat ongekende kenmerken geen probleem vormen.
CR-41		30 Jan 2017	Classification: Factually incorrect Ranking: Major	'k zou zeggen CTRL+Z maar een paar keer
CR-50		30 Jan 2017	Ranking: Minor Classification: Not conforming to standards	#dim IngegevenKenmerken As ??????
CR-50		30 Jan 2017		Lijkt mij beter om aan de DOM.PM.Maatwerk.IngegevenKenmerkenTypeAPI een method toe te voegen ala IsTandemboxLade() op voorhand dat de ingegevenkenmerken van het type DOM... zijn
CR-50		30 Jan 2017		idem als hierboven
CR-50		30 Jan 2017		Als we dan toch bezig zijn

Atlassian FishEye analysis with Crucible code review. (Version:4.2.1 Build:20161109135523 2016-11-09) - Administration - Page generated 2025-09-02 05:02 +0200