Redmine

Hi,

When editing Spent Time the current time entry user is shown twice in the User drop-down.

This is because the condition to add the current time entry user to the collection is wrong in TimelogHelper#user_collection_for_select_options.

It is:

collection << time_entry.user unless time_entry.user.nil? && !collection.include?(time_entry.user)

But it should be:

collection << time_entry.user if time_entry.user && !collection.include?(time_entry.user)

Best regards,
Thomas