Fix infinite loop on cloning environment group with another clone present
ClosedPublic

Authored by kossebau on Jan 4 2017, 1:24 PM.

Details

Summary

Clone numbering was broken and fixed to the number 1,
because the argument was already resolved initially.

Test Plan
  1. Clone existing environment group
  2. Clone the same environment group a second time

Without this patch an infinite loop is entered.

Diff Detail

Repository
R33 KDevPlatform
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
kossebau updated this revision to Diff 9703.Jan 4 2017, 1:24 PM
kossebau retitled this revision from to Fix infinite loop on cloning environment group with another clone present.
kossebau updated this object.
kossebau edited the test plan for this revision. (Show Details)
kossebau added a reviewer: KDevelop.
Restricted Application added a subscriber: kdevelop-devel. · View Herald TranscriptJan 4 2017, 1:24 PM
kfunk accepted this revision.Jan 4 2017, 3:53 PM
kfunk added a reviewer: kfunk.
kfunk added a subscriber: kfunk.

LGTM

Let me push that to 5.0 branch

This revision is now accepted and ready to land.Jan 4 2017, 3:53 PM
This revision was automatically updated to reflect the committed changes.