It feels off not using existing logic for the none `sublist_fields` attributes. Wouldn't it be better to fork the logic around line 295, or maybe even within the `populateLineItem` method?