Title processing changes

[munnja001]

Fixed two bugs which make the inheritance of titles much more natural

1st: If the content did not have a "head" tag, the decorator title would not be processed leaving the title blank. I updated the code to properly process the decorator title when the content head is null

2nd: When either the decorator title or content title are empty, the title pattern is being processed which is resulting in unnatural titles such as "Decorator Title -" instead of "Decorator Title". I updated the code to only process the title pattern if both decorator title and content title are not empty

Thanks for the great plugin!

• Justin

[ultraq]

Soooooo glad you removed the reliance on Apache's StringUtils - I am not a fan of that library (have been stung by it in the past w/ performance critical code).

Seeing all the 'null or empty' checks though, maybe one day I'll rewrite the thing in Groovy just to keep the whole thing cleaner-looking!

Anyway, I'm pretty happy with the changes and the general idea behind them. I'll merge them when I get some time to give it all a closer look, hopefully with any fixes to bug reports that might come through, although I'd be happy to release a minor version with just this change too.

[ultraq]

This change is now in version 1.2.4, which was released today.