No abbreviation for 'units'

I'm sorry it's taken me so long to get to testing all of these new patches. I'll try to get through them when I can to help out.

This issue was of particular interest to me. In the recent project I was working on that involved Recipe, the client asked me to remove these blank abbreviations too. I haven't yet been able to find a reason why they're displayed like this.

The patch in the OP had 1 whitespace error, which was easy to fix. It also didn't account for the other blank abbreviation unit, "unknown." I added the unit. Then I took the opportunity to clean up the function a bit, including parts of it that seemed leftover from earlier versions of the module.

I don't mean to take away from Grayside's attempt to patch the issue and it's also possible that I did too much. If I need to re-roll the patch and scale it back, I can.

Go, testbot.

Here's another patch which doesn't try to do so much.

That's funny, I was just looking at this.
Let's compare

I think I like yours better, ie using the empty abbrev to key off of.
Do you run the unit tests before posting?

Yeah, I checked yours and I see what you're trying to do, but I prefer mine too.

I did run the tests in this case because I knew the patch would break a few asserts since there were some specifically checking for the empty abbr tag. I won't for every patch. It takes a long time to run all of the tests locally. I didn't time it or pay close attention, but I think it was about 15min. Testbot, however, can run them in just a minute or two.

#5: 1590220-5-empty-unit-abbreviation.patch queued for re-testing.

Patch #5.

#5 has been committed to 7.x-1.x.