Ilogeek / redmine_issue_dynamic_edit

Allows users to dynamically update (AJAX) issue's attributes from issue details page without any refresh (JIRA like)
http://www.redmine.org/plugins/redmine_issue_dynamic_edit
MIT License
134 stars 56 forks source link

The plugin modifies the AJAX requests outside the ones it is handling by itself #119

Closed issrec closed 9 months ago

issrec commented 9 months ago

The plugin causes all AJAX requests to be implemented as Accept: text/html.... This may lead to impaired cooperation with other solutions that introduce their own (sub)forms. The plugin does not work (is not compatible) with:

This is most likely due to placing all edit fields in the main/single form with id: fakeDynamicForm.

Case analysis and diagnosis of the problem in the external task: HTTP error 406 during recurrence creation (text/html format requested)

Ilogeek commented 9 months ago

https://github.com/Ilogeek/redmine_issue_dynamic_edit/commit/f4f2862db708b6b0583679d6d3e30e299e7dbbbe#diff-db9ffa4425160ce239c4587a84f05e4fccf24e53577811f129b919d05e655e54L128 removed fakeDynamicForm form field

I do not know how Issue recurring performs its call but it may be not prevent the default behavior of submitting data located into a form field. I can not find anywhere into the Issue dynamic edit codebase anything altering all AJAX call.

Don't hesitate to tell me if Issue recurring still create conflict with Issue dynamic edit even after the last commit

issrec commented 9 months ago

I thank you for the corrections you made. After initial testing, it seems that the problem of implementing all AJAX requests as Accept: text/html... no longer occurs. Additionally, I was able to re-enable the _CONF_CHECK_ISSUE_UPDATE_CONFLICT option.

I noticed another incompatibility of both plugins, but it seems to be a different thread, so I registered a separate issue - #121.

Thank you here, the problem seems to be solved.