Patch #26312

Remove duplicate notifiable require

Added by Mischa The Evil 4 months ago. Updated 4 months ago.

Status:ClosedStart date:
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Code cleanup/refactoring
Target version:-

Description

I noticed this recently. The require 'redmine/notifiable' at source:/trunk/lib/redmine.rb#L42 is duplicated at source:/trunk/lib/redmine.rb#L45. This was introduced in r10909

I'll leave a patch against trunk removing the duplicate line.

0001-Remove-duplicate-notifiable-require.patch Magnifier (713 Bytes) Mischa The Evil, 2017-07-02 09:01

Associated revisions

Revision 16736
Added by Toshi MARUYAMA 4 months ago

Remove duplicate notifiable require (#26312).

This duplication was introduced as part of r10909.

Contributed by Mischa The Evil.

Revision 16737
Added by Toshi MARUYAMA 4 months ago

Merged r16736 from trunk to 3.4-stable (#26312).

Remove duplicate notifiable require.

This duplication was introduced as part of r10909.

Contributed by Mischa The Evil.

History

#1 Updated by Toshi MARUYAMA 4 months ago

  • Status changed from New to Closed
  • Target version deleted (3.4.0)

Committed in trunk and 3.4-stable, thanks.

#2 Updated by Mischa The Evil 4 months ago

Toshi: thanks for the quick commit.

Also available in: Atom PDF