KiBoM icon indicating copy to clipboard operation
KiBoM copied to clipboard

Merge blank fields logic

Open set-soft opened this issue 4 years ago • 0 comments

The current logic is:

https://github.com/SchrodingersGat/KiBoM/blob/06a14866d8166cfa71ae8ac1a48bda0c160fb380/kibom/component.py#L98-L100

I don't get it. If the idea is to loosly group fields: should't we be doing a return True? I mean:

 if this_field == "" or other_field == "": 
     if self.prefs.mergeBlankFields: 
         return True

Note that even when the condition seems to be equivalent it isn't. In this version of the code we have a chance to return True, in the other don't.

set-soft avatar Aug 11 '20 15:08 set-soft