Recursive dependencies should be detected at template saving and forbidden

When there is a recursive dependency in a template (like if you put %sn% in uid and %uid% in sn), the error message is not clear enough.

(from redmine: issue id 4840, created on 2016-06-03, closed on 2017-01-17)

• Changesets:
  • Revision 9b11b2c8 by Côme Chilliet on 2016-06-03T06:48:58.000Z:

Fixes #4840 Improving recursion dependency detection and behavior

• Revision ba318002 by Côme Chilliet on 2016-06-03T06:49:41.000Z:

Fixes #4840 Improving recursion dependency detection and behavior

• Revision 3fb22d8b by Côme Chilliet on 2016-06-03T09:03:44.000Z:

Fixes #4840 fixed an error in recursive dependency error message

• Revision d64a32e2 by Côme Chilliet on 2016-06-03T09:04:58.000Z:

Fixes #4840 fixed an error in recursive dependency error message

• Revision 93b8c263 by Côme Chilliet on 2016-10-10T08:50:36.000Z:

Fixes #4840 Detecting recursive dependencies at template save

• Revision b5724e09 by Côme Chilliet on 2017-01-17T11:34:16.000Z:

Fixes #4840 Fixed recursive dependency detection in templates

• Custom Fields:
  • Bug in version: 1.0.12

We need to test this, and maybe add a test case (in csv import most probably, through using the template directly should produce the same result).

(from redmine: written on 2016-06-03)

By Côme Chilliet on 2017-09-02T15:29:51 (imported from GitLab)

The error message is now clear enough.

(from redmine: written on 2016-06-03)

By Jonathan Swaelens on 2017-09-02T15:29:51 (imported from GitLab)

It’s better but I think we should find a way to detect and forbid recursive dependencies when saving the template (so, in the check method most probably… but maybe in the tab class as dependencies can go across tabs…)

(from redmine: written on 2016-06-03)

By Côme Chilliet on 2017-09-02T15:29:51 (imported from GitLab)

This can be done later, letting this open to keep it in mind.

(from redmine: written on 2016-06-07)

By Côme Chilliet on 2017-09-02T15:29:51 (imported from GitLab)

It should now be detected on template save. The same mechanism could be used to detect other template syntax problems (unknown modifiers and so on)

(from redmine: written on 2016-10-10)

By Côme Chilliet on 2017-09-02T15:29:51 (imported from GitLab)

Close issue

(from redmine: written on 2017-01-17)

By Côme Chilliet on 2017-09-02T15:29:52 (imported from GitLab)

closed

By Côme Chilliet on 2017-09-02T15:29:52 (imported from GitLab)

added Fixed label

By bmortier on 2018-09-24T14:06:40 (imported from GitLab)

removed Bugs label

added PJ1802-0188 fusiondirectory-core labels