Must be missing something .... Not sure I understand why we are deleting items and reentering them ...... wouldn't we lose history on when they were originally put in if we do that?
this line is: cbGridAnnoType.DataSource = AnnotationTypeInfoList.Get().Clone();
2-4 lines below have:
cbGridAnnoType.DataSource = VEPROMS.CSLA.Library.AnnotationstypeSelections.Get(MyUserInf…
this should be removed if data set is set to something else (see 2 lines after this)
unless reason for keeping the commented out code (should be explained in the comments) --- commented out code shouldn't be checked in?
commented out code should be removed or reason given for why it is commented out / should be in the code long term.
just to double check --- Is there a reason we are updating the DisplayMember / ValueMember every time we go through the loop?
just to check - is there a reason these 2 items (the id and NameStr) are global variables / should stay in memory while the form is open?
commented out code should be removed or reason given for why it is commented out / should be in the code long term.
commented out code should be removed or reason given for why it is commented out / should be in the code long term.
unless reason for keeping the commented out code (should be explained in the comments) --- commented out code shouldn't be checked in?
Now, this is a method with everything inside commented out --- if no code is to execute, should the method be removed?
commented out code should be removed or reason given for why it is commented out / should be in the code long term. (lines 122-127)