[ICT-1976] PM: MaatwerkProduct heeft geen IngegevenKenmerken (cfr. SST-LBX)

Activity

CR-2255 1

Keyboard shortcuts  
  • Summarize the review outcomes (optional)
     
    #permalink

    Details

    Warning: no files are visible, they have all been filtered.
    Participant Role Time Spent Comments Latest Comment
    Author & Moderator 4m 1 Reden van deze wijzigingen : method .GeefProductSpecifica...
    Reviewer - Complete 4m    
    Total   8m 1  
    #permalink

    Objectives

    There are no specific objectives for this review.

    Branches in review

    #permalink

    Issues Raised From Comments

    Key Summary State Assignee
    #permalink

    General Comments

    Wim Vermeulen

    Reden van deze wijzigingen : method .GeefProductSpecificatie() kan antwoorden...

    Reden van deze wijzigingen :
    method .GeefProductSpecificatie() kan antwoorden met een leeg object.
    Wanneer men hierop meteen .GeefIngegevenKenmerken() oproept, crasht caché met een "InvalidOref" en geen enkele notie over welk product het gaat.
    In de rechtstreekse implementatie van .GeefIngegevenKenmerken() is deze fout wel goed opgevangen, en zal een leesbare foutmelding gesmeten worden.

    /APPS/Halux/.../LBX/BladInhoudBuilder.cls.xml Changed
    /APPS/Halux/.../TAX/BladInhoudBuilder.cls.xml Changed
    Open in IDE #permalink
    /APPS/.../Comparator/LBXComparator.cls.xml Changed
    Open in IDE #permalink
    /APPS/.../Comparator/MVXComparator.cls.xml Changed
    /APPS/.../ProductieService/ProductIDCategorieBepaler.cls.xml Changed
    /DOM/PM/.../impl/LBXLadeImpl.cls.xml Changed
    Open in IDE #permalink
    /DOM/PM/.../impl/MVXLadeImpl.cls.xml Changed
    /DOM/PM/.../impl/TAORLadeImpl.cls.xml Changed
    /DOM/PM/.../DataM/DataMMaatwerkProduct.cls.xml Changed
    /DOM/PM/impl/MaatwerkProductImpl.cls.xml Changed
    /vhUnitTest/.../BepaalCategorie/Base.cls.xml Changed
    Open in IDE #permalink

    Review updated: Reload | Ignore | Collapse

    You cannot reload the review while writing a comment.

    Log time against