I think this is a bug. Once you have a saved log with a quantity adjustment against an inventor asset, you cant change the inventory asset. There is a delete button to remove it but when you click the select asset button it does nothing.
1 Like
Oops yea that does look like a bug. Thanks for the report @BOTLFarm!
That button is provided by the Entity Browser module, so that’s where the investigation probably begins…
If you click “Update quantity” and then edit it again, does it work?
And do you see any errors in the browser console when you trigger the bug?
No errors in the console that I see.
Yes if I remove the asset, then click update quantity, then I can add a new asset.
1 Like
I just replicated this on both farmOS 3.x and 4.x.
Notably, mine is similarly broken, but there is one slight difference: when I click the trash icon next to the asset (like in your screencapture), the entire “Inventory asset” box disappears, including the “Select assets” button, and changing the “Inventory adjustment” type radios doesn’t make it reappear.
So something’s definitely up… this might not be a new issue… you might just be the first to report it @BOTLFarm. 
Found these two open issues in the Entity Browser module queue that sound similar:
1 Like
I’ll take this opportunity to say: I’ve always disliked the Entity Browser’s “tiles w/ trash can” widget style. It’s different than all other standard Drupal design patterns, which makes it confusing to use… and in this case buggy!
I recall talking about this with @paul121 and daydreaming about replacing the widget with something simpler. The popup for selecting entities itself is OK (a little overwhelming IMO), but the widget that shows the selected entities could be a low-hanging opportunity for improvement. And that’s not too hard because field widgets are plugins.
I’ve also had larger ambitions of reworking all of our “entity referencing” form widgets everywhere in farmOS… maybe I need to write up a forum topic to get that ball more formally rolling.
Given that there is a workaround, I probably won’t dedicate time to investigating this.
This feels like a better use of time IMO.
Completely agree, this is workable as is!!
1 Like