Comment Search

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

Search Criteria

Project
Comment content
Review PermaId
After
Before
Comment Author
Search Type
Search Type:
Review State
Review State:
Metrics:
Metric Ranking
Metric Classification
Submit
  •  

Comment Results

Review Name Created Custom Fields Content
CR-227 17 May 2017

Aangezien dit de clou is van de test, kunt ge er misschien beter een ExactAantalKeer(1) bijzetten?
CR-227 17 May 2017

Misschien beter om de testen hier op te splitsen per method in de Service en dan misschien hier en daar nog een testje bijgooien om wat randgevallen of true-false dingen af te toetsen
CR-200 17 May 2017

is inderdaad nadien gewijzigd in UST2416
CR-227 17 May 2017

beter de vhTest.Fake.TECH.Mail ?
CR-200 17 May 2017

is dezelfde item, maar oude deploy wordt overbodig omdat item nieuwe waarde krijgt. Of bedoel ge iets anders ?
CR-227 17 May 2017

Beter een ExactAantalKeer(0) op de MailAPIMock, om duidelijk de intentie te tonen
CR-227 17 May 2017

Wat met klanten zonder transportkeuze? Of is dat niet relevant?
CR-227 17 May 2017

ExactAantalKeer(1) om intentie duidelijk te maken
CR-227 17 May 2017

Wat met klanten zonder transportkeuze?
CR-228 17 May 2017

Waarom die nieuwe parameter? Er wordt op het eerste zicht niets mee gedaan.
CR-228 17 May 2017

$$$Not ipv '
Caché kennende is het ook veiliger om if (()&&()) te doen ipv if () && () (ook hieronder)
CR-228 17 May 2017

hier en daar wat casing aanpassen
CR-228 17 May 2017 Ranking: Minor Classification: Not conforming to standards

Moet eigenlijk in /vhUnitTest/.../Service/BerekenPalletGegevens/Test.cls
CR-228 17 May 2017 Ranking: Minor Classification: Not conforming to standards

Moet eigenlijk in /vhUnitTest/.../Service/GeefMogelijkeZendingen/Test.cls
CR-215 17 May 2017 Classification: Editorial Ranking: Minor

FOP ???? en Casing

CR-228 17 May 2017

Het ging hier over een bestaande parameter zonder type. ik heb het type toegevoegd.
CR-215 17 May 2017 Ranking: Minor Classification: Improvement desirable

Ik zou eerder opteren om de PrintEtiket method te gebruiken met de Device en de andere te renamen naar PrinterEtiketViaComputer of zelfs deze functionaliteit in een andere klasse te plaatsen want het ophalen van de printer via de computernaam heeft opzich niks te maken met het bepalen van de printer voor een bepaalde computernaam voor een bepaald etiket.
CR-227 17 May 2017

aangezien deploy wordt er geen aanpassing doorgevoerd.
CR-226 17 May 2017 Classification: Missing Ranking: Major

Hiervoor zou ik wel een specifieke test maken zodoende we zeker zijn dat die SpecifiekeData goed doorkomt tot in de UserInterface.

Test : Met prijsverschil geval en test die melding heeft dat order reeds besteld is en test dat er een andere fout is opgetreden
CR-230 17 May 2017 Classification: Inconsistent Ranking: Minor

Als je bovenaan werkt met het +PersonalityId systeem zou ik dat hier ook doen
CR-230 17 May 2017 Ranking: Minor Classification: Not testable

%OpenId vervangen door Repo
CR-230 17 May 2017 Classification: Inconsistent Ranking: Minor

+Personality ???
CR-227 17 May 2017

Is ook gecovered door Method "Test: BereidVoorBestelling plaatst de hoofding op de offerte en bepaalt de mogelijke leverdatums - standaard zijn er geen herberekende lijnen en meldingen aan de gebruiker"()
Ik heb in ieder geval een test toegevoegd die controleert dat er geen verzendingmogelijkheden berekend worden voor standaard klanten.
CR-227 17 May 2017

zie hierboven
CR-222 17 May 2017 Classification: Improvement desirable Ranking: Major

Code verplaatsen naar waar de andere controle wordt uitgevoerd.. zodat de beslissing op de juiste plaats staan
CR-226 17 May 2017

manueel getest, maar er is intussen ook al een nieuw kaartje om dit te herwerken naar deftige implementatie voor intracto. (momenteel bezig.)
CR-225 17 May 2017 Classification: Factually incorrect Ranking: Major

Geen constructor parameters meegeven aan de logger , dit maakt de groep van de logging onbruikbaar
CR-225 17 May 2017 Ranking: Major Classification: Factually incorrect

Externe id vermelden want dat zal in de toekomst de enigste nummer zijn die OV zal kennen.
CR-229 17 May 2017 Ranking: Major Classification: Improvement desirable

Beter om dit in één of andere WSimpl.Vhisie4.SettingsManager af te handelen ipv de personalities te vervuilen met specifieke vhisie4 logica
CR-162 17 May 2017 Ranking: Minor Classification: Improvement desirable

$$$If structuur gebruiken..
CR-162 17 May 2017 Classification: Editorial Ranking: Minor

Response uitbouwen op het einde van de code..

Niet we doen iets .. we bouwen den Response op en we doen nog iets en we quiten de response.

Beter we doen iets , we doen nog iets , we bouwen de response op en we quiten die
CR-179 17 May 2017 Ranking: Minor Classification: Editorial

Het is te hopen dat de andere code dat ook doet !?

Gelieve hiervoor een betere omschrijving te bedenken. Een korte samenvatting van het wat gedrag is m.a.w. en niet hoe maar wat er juist gedaan wordt.
CR-193 17 May 2017 Ranking: Minor Classification: Extra (superfluous)

Cleanup
CR-193 17 May 2017 Ranking: Minor Classification: Improvement desirable

GeefVhisie3OfferteLink
CR-193 17 May 2017 Ranking: Major Classification: Extra (superfluous)

Geef Of Verwerk en niet GeefVerwerk

CR-193 18 May 2017

klopt wel zn. Het is de link om een vhisie4 of vhisie3 offerte te verwerken. Dus verwerkOffertelink en als je die geeft via een method is het geefverwerkvhisie4OfferteLink.
(Het is niet het verwerken van vhisie4 dat uitgevoerd wordt in de code.)
CR-233 19 May 2017

casing trekt op niets en methodname/testname is niet altijd correct
voor de rest dik in orde
CR-236 22 May 2017

Aangezien de meeste commits in deze story van mezelf zijn, valt er niet veel te vertellen
CR-241 22 May 2017 Classification: Cleanup Ranking: Major

Storage ook in het oog houden als er properties worden gewijzigd
CR-241 22 May 2017 Classification: Editorial Ranking: Major

gelieve deze tekst toch iets duidelijker te maken voor de gebruiker :

want 'k weet niet of ze het begrip terminalnavigatie allemaal gaan kennen.

Iets duidelijker zou kunnen zijn :

Order 'xxxxx' is gemaakt. Er was echter een probleem om admin automatisch op te starten. Gelieve manueel admin te starten om zo het order verder te verwerken.
CR-240 22 May 2017 Ranking: Minor Classification: Improvement desirable

WriteLine best vervangen door Write, kwestie van geen overbodige /// in de gegenereerde klasses te krijgen. Zie verwijderde lijn waar zo'n Write gebruikt wordt.
CR-240 22 May 2017 Classification: Inconsistent Ranking: Minor

' -> $$$Not
CR-240 22 May 2017 Ranking: Minor Classification: Cleanup

wlipkes zijn niet meer nodig nadat ze gebruikt worden om problemen op te lossen
CR-241 22 May 2017 Classification: Editorial Ranking: Minor

..Matches.Object(GeefGegevenBevestigingen)
CR-241 22 May 2017 Classification: Editorial Ranking: Minor

Er wordt een melding aan de gebruiker gegenereerd als de terminal navigatie niet kan gestart worden
CR-243 22 May 2017 Classification: Improvement desirable Ranking: Major

Eigenlijk geen goed idee om de GeefWebshopVerwerker op de personality te plaatsen.

Beter ergens op een klasse te zetten die dichter bij de webshop staat en daar de verwerker te vragen voor een bepaalde personality

Het domein dient niet te weten dat er ergens een webshop bestaat.
CR-246 23 May 2017 Ranking: Minor Classification: Cleanup

Hebben we hier niet wat veel duplicate code ,

Misschien beter om ergens een private method te maken : alla AssertDatErGeenMailVerstuurdWordt
CR-240 23 May 2017 Classification: Factually incorrect Ranking: Major

Let erop dat deze niet de de methods van de huidige klasse meeneemt!!!
CR-229 23 May 2017 Classification: Improvement desirable Ranking: Major

HandelklantDataRepo en KlantDataBewaarder wel nodig ?? Kan de een de ander niet oproepen ??
CR-229 23 May 2017 Classification: Extra (superfluous) Ranking: Major

Deze lijnen kunnen die niet in de SessionManager ???