Project

General

Profile

Bug #3753

It is possible to create a new Directive with empty values in all its variables from the API

Added by Nicolas PERRON over 4 years ago. Updated 10 months ago.

Status:
Rejected
Priority:
1
Assignee:
-
Category:
API
Target version:
Target version (plugin):
Severity:
User visibility:
Effort required:
Pull Request:
Priority:

Description

With the new API, it is possible to create a new Directive but it is also possible to create a new Directive without any variable set.

The result is a Directive disabled and not possible to enable since its mandatory variables are empty. I'm not sure about this behavior.

The good point is that it does not break anything. What's your opinion, Jon ?

History

#1 Updated by Jonathan CLARKE over 4 years ago

  • Assignee changed from Jonathan CLARKE to Nicolas PERRON

Nicolas PERRON wrote:

With the new API, it is possible to create a new Directive but it is also possible to create a new Directive without any variable set.

The result is a Directive disabled and not possible to enable since its mandatory variables are empty. I'm not sure about this behavior.

The good point is that it does not break anything. What's your opinion, Jon ?

This doesn't sound unreasonable, in principle. I can imagine a scenario where you want to pre-create Directives, just knowing their name and Technique/version, but don't want to fill in their variables yet.

However, this shouldn't cause the user to "get stuck". If you can create this Directive, and then also modify it (both by the API and the WebUI), in order to be able to fill it in and then enable it, I'm fine with that. Nico, can you confirm this is possible?

Otherwise, we need to provide a solution to "get out" of this corner.

#2 Updated by François ARMAND about 3 years ago

I can confirm that correctly seting the mandatory parameters of the directive allow to save it and enable it.

#3 Updated by Benoît PECCATTE over 2 years ago

  • Assignee deleted (Nicolas PERRON)
  • Target version set to 2.10.14

#4 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.14 to 2.10.15

#5 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.15 to 2.10.16

#6 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.16 to 2.10.17

#7 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.17 to 2.10.18

#8 Updated by Vincent MEMBRÉ over 2 years ago

  • Target version changed from 2.10.18 to 2.10.19

#9 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.10.19 to 2.10.20

#10 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.10.20 to 2.11.18

#11 Updated by Vincent MEMBRÉ about 2 years ago

  • Target version changed from 2.11.18 to 2.11.19

#12 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 2.11.19 to 2.11.20

#13 Updated by Vincent MEMBRÉ almost 2 years ago

  • Target version changed from 2.11.20 to 2.11.21

#14 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 2.11.21 to 2.11.22

#15 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 2.11.22 to 2.11.23

#16 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 2.11.23 to 2.11.24

#17 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 2.11.24 to 308

#18 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 308 to 3.1.14

#19 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.14 to 3.1.15

#20 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.15 to 3.1.16

#21 Updated by Vincent MEMBRÉ over 1 year ago

  • Target version changed from 3.1.16 to 3.1.17

#22 Updated by Vincent MEMBRÉ about 1 year ago

  • Target version changed from 3.1.17 to 3.1.18

#23 Updated by Vincent MEMBRÉ 11 months ago

  • Target version changed from 3.1.18 to 3.1.19

#24 Updated by Benoît PECCATTE 11 months ago

  • Subject changed from It is possible to create a new Directive with empty values in all its variables to It is possible to create a new Directive with empty values in all its variables from the API
  • Status changed from New to Pending technical review

#25 Updated by Benoît PECCATTE 11 months ago

  • Status changed from Pending technical review to New

#26 Updated by François ARMAND 10 months ago

  • Status changed from New to Rejected

Closing the ticket given the comments.

Also available in: Atom PDF