[VELOCITY-560] Should ExtendedProperties.convertProperties instead of ExtendedProperties.putAll

• Type: Bug
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 1.5
• Fix Version/s: 1.6
• Component/s: Documentation
• Environment: Any
• Assignee: Unassigned
• Reporter: Alik Eliashberg
• Created: 02/Aug/07 19:25
• Updated: 04/Sep/08 06:26
• Resolved: 04/Sep/08 06:26

Description

In the developer-guide document (http://velocity.apache.org/engine/releases/velocity-1.5/developer-guide.html), there is a section, which talks about using ExtendedProperties to initialize VelocityEngine (at the bottom of #miscellaneousdetails).

At least for me, it didn't work as described. Instead of:
ExtendedProperties eprops = new ExtendedProperties();

// if you already have a Properties object do this
eprops.putAll(props);

It should be:
ExtendedProperties eprops = null;
if (props==null)

{ eprops = new ExtendedProperties(); }

else

{ eprops = convertProperties(props); }

This is obviously very minor, but may save someone 15 minutes.