AJAX error when editing term content

@kyuubi, Please review https://www.drupal.org/project/taxonomy_manager/issues/1120178 for the same, hope this will helps you.

thanks

I see this behaviour as well, those fields post to the TaxonomyManagerForm instead of their own form. I think the solution could be that the TaxonomyManagerForm has to proxy it to the right form or the AJAX post has to be done to the right form.

Any update on this issue?

We are also using this module and not able to update/create the referenced Media Entities.

Thanks @kyuubi for work around.
As currently term page is open in current tab. So after save or clicking back button tree structure removed because of page load. So, I am re-rolling your patch with small update to open term page in new tab.

Can you verify it is still an issue in 2.0 branch?

@VladimirAus,

I tested and the issue doesn't happen on the 2.0 version.

However, when using it along with term_reference_fancytree module, the fancytree doesn't work as expected with multi level taxonomy terms. I assume this is not a taxonomy manager issue, right?

Probably need to debug the term_reference_fancytree code to find out why the bug happens.

Good idea to create separate bug for term_reference_fancytree

I'm reopening this bug ticket as I'm getting exactly the same issue on version 2.0.8

@alfattal if you are reopening the bug, could you please provide

• versions of your environment
• screenshots
• steps to reproduce

@VladimirAus Sorry for the late response. Here is the requested information:

Version of my environment:
Drupal 9.5.2
PHP 8.0.27
Taxonomy Manager version 2.0.7 and 2.0.8

Steps to reproduce:

1. Go to Structure > Taxonomy and edit any vocabulary.
2. In the Manage fields tab, add general field like (Email) or (Link)
3. In the new field setting, select Unlimited, save field settings then save settings. This will add the Add another item button
4. Navigate to the list tab and add some terms to your vocabulary
5. Go to Structure > Taxonomy Manager and click on that vocabulary. From the terms list block on the left, click on any term. This will open the edit panel to the right.
6. Scroll down to the field you added and click on the Add another item. A spinning (Please wait) will show momentarily and then disappear without adding another field.

Screenshots:

Here is the AJAX error on Chrome console:

Full trace:

{
    "message": "\nAn AJAX HTTP error occurred.\nHTTP Result Code: 200\nDebugging information follows.\nPath: /admin/structure/taxonomy_manager/voc/tags?ajax_form=1&_wrapper_format=drupal_ajax\nStatusText: parsererror\nResponseText: \nTaxonomy Manager - Tags | Drush Site-Install\nSkip to main content\nToolbar items\nBack to site\nManage\nAdministration menuContentStructureAppearanceExtendConfigurationPeopleReportsHelp      \nShortcuts\nUser-defined shortcutsAdd contentAll contentEdit shortcuts      \nEdit\nTour\nadmin\nUser account actionsView profileEdit profileLog out      \nBreadcrumb\nHome\nAdministration\nStructure\nTaxonomy Manager\nTaxonomy Manager - Tags\nAdd to Default shortcuts\nToolbar\nTags\nSearch terms\nLoading…\nTags\n{\"ajaxPageState\":{\"theme\":\"claro\",\"theme_token\":\"OLKnMVjhWyiImDCxpIkb1IoTy1hcl-TecPwcGl9KVNQ\",\"libraries\":\"big_pipe\\/big_pipe,ckeditor5\\/internal.drupal.ckeditor5,ckeditor5\\/internal.drupal.ckeditor5.codeBlock,ckeditor5\\/internal.drupal.ckeditor5.emphasis,ckeditor5\\/internal.drupal.ckeditor5.htmlEngine,ckeditor5\\/internal.drupal.ckeditor5.image,claro\\/global-styling,contextual\\/drupal.contextual-links,contextual\\/drupal.contextual-toolbar,core\\/ckeditor5.basic,core\\/ckeditor5.blockquote,core\\/ckeditor5.essentials,core\\/ckeditor5.horizontalLine,core\\/ckeditor5.htmlSupport,core\\/ckeditor5.link,core\\/ckeditor5.list,core\\/ckeditor5.pasteFromOffice,core\\/ckeditor5.removeFormat,core\\/ckeditor5.sourceEditing,core\\/ckeditor5.table,core\\/drupal.active-link,core\\/drupal.autocomplete,core\\/drupal.collapse,core\\/drupal.message,core\\/drupal.states,core\\/drupal.tabledrag,core\\/drupal.tableresponsive,core\\/internal.jquery.form,core\\/normalize,filter\\/drupal.filter,shortcut\\/drupal.shortcut,system\\/admin,system\\/base,taxonomy_manager\\/form,taxonomy_manager\\/tree,toolbar\\/toolbar,toolbar\\/toolbar.escapeAdmin,tour\\/tour,user\\/drupal.user.icons\"},\"ajaxTrustedUrl\":{\"form_action_p_pvdeGsVG5zNF_XLGPTvYSKCf43t8qZYSwcfZl2uzM\":true,\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\":true},\"ajax\":{\"edit-load-term-data--kxdjNpgrWLU\":{\"callback\":\"::termDataCallback\",\"event\":\"change\",\"url\":\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\",\"dialogType\":\"ajax\",\"submit\":{\"_triggering_element_name\":\"load-term-data\"}},\"edit-miniexport--kxXyNbRWTjU\":{\"callback\":\"::exportListFormCallback\",\"event\":\"mousedown\",\"keypress\":true,\"prevent\":\"click\",\"url\":\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\",\"dialogType\":\"ajax\",\"submit\":{\"_triggering_element_name\":\"export\",\"_triggering_element_value\":\"Export all\"}},\"edit-export--fRfwxOOYXAc\":{\"callback\":\"::exportFormCallback\",\"event\":\"mousedown\",\"keypress\":true,\"prevent\":\"click\",\"url\":\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\",\"dialogType\":\"ajax\",\"submit\":{\"_triggering_element_name\":\"export\",\"_triggering_element_value\":\"Export\"}},\"edit-move--jwU73IX3erQ\":{\"callback\":\"::moveFormCallback\",\"event\":\"mousedown\",\"keypress\":true,\"prevent\":\"click\",\"url\":\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\",\"dialogType\":\"ajax\",\"submit\":{\"_triggering_element_name\":\"move\",\"_triggering_element_value\":\"Move\"}},\"edit-delete--fynBIP6l8gY\":{\"callback\":\"::deleteFormCallback\",\"event\":\"mousedown\",\"keypress\":true,\"prevent\":\"click\",\"url\":\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\",\"dialogType\":\"ajax\",\"submit\":{\"_triggering_element_name\":\"delete\",\"_triggering_element_value\":\"Delete\"}},\"edit-add--cqI4DM4ijJU\":{\"callback\":\"::addFormCallback\",\"event\":\"mousedown\",\"keypress\":true,\"prevent\":\"click\",\"url\":\"\\/admin\\/structure\\/taxonomy_manager\\/voc\\/tags?ajax_form=1\\u0026_wrapper_format=drupal_ajax\",\"dialogType\":\"ajax\",\"submit\":{\"_triggering_element_name\":\"add\",\"_triggering_element_value\":\"Add\"}}},\"taxonomy_manager\":{\"tree\":[{\"id\":\"edit-taxonomy-manager-tree--pwJAKCLpM04\",\"name\":\"taxonomy[manager][tree]\",\"source\":[{\"title\":\"Blue\",\"key\":\"1\"},{\"title\":\"Green\",\"key\":\"3\"},{\"title\":\"Red\",\"key\":\"2\"}]}]},\"toolbar\":{\"breakpoints\":{\"toolbar.narrow\":\"only screen and (min-width: 16.5em)\",\"toolbar.standard\":\"only screen and (min-width: 38.125em)\",\"toolbar.wide\":\"only screen and (min-width: 61em)\"},\"subtreesHash\":\"7-9vto32uNdmH-YRS0uRMvyrDqozLqY623gYlczNClc\"},\"path\":{\"baseUrl\":\"\\/\",\"pathPrefix\":\"\",\"currentPath\":\"admin\\/structure\\/taxonomy_manager\\/voc\\/tags\",\"currentPathIsAdmin\":true,\"isFront\":false,\"currentLanguage\":\"en\",\"currentQuery\":{\"_wrapper_format\":\"html\",\"ajax_form\":\"1\"}},\"pluralDelimiter\":\"\\u0003\",\"user\":{\"uid\":\"1\",\"permissionsHash\":\"ed2403a2ae29a27933ef8500c439ca84cc2c30ae5ee88c6dfc645572278c4bb4\"}}\n",
    "name": "AjaxError"
}

@VladimirAus Did you have the chance to look into the information I provided?

hi all
i also confirm this issue after selecting "Entity browser" as Widget on field's form display and now going back to the edit/add term page

Please create MR.

I also have the same scenario as @delacosta456 and have confirmed that it is happening.

It looks like it's related to the Drupal core issue https://www.drupal.org/project/drupal/issues/2504115.

The workaround is to render the form without Ajax and send a taxonomy term ID through the URL query parameter.
The patch is attached.

I couldn't expand the tree to the given TID using the 'terms_to_expand' setting. Probably, maintainers could help with it.

Thank you for your contribution.
Can you create MR?

@dburiak, I pushed your patch to the branch. Please fix the issues we found while testing the latest changes.

The PR is updated.
The updated patch can be found in the attachment.

I had to make a small adjustment to this patch since it did not apply cleanly due to another taxonomy manager patch that I had to apply earlier. Here is an updated patch just in case there are folks out that applied patch from issue 3474919.

created a patch from MR (it's merged with #3474919)
make sure library/jquery.fancytree contains all JS - see #3467549

Thank you all! Committed! 🍻
This issue also fixes #3474919: Fix broken form element taxonomy manager tree.

This issues causes the toolbar position to be next to the tree and only abote the term page. Before it was above both the tree and term page. See images.

Before:

After:

The search functionality is also not working with these changes..

It also seems rather radical to completely render the form without Ajax, as this is not ideal from a user experience perspective. Do the maintainers have any plans to look into a solution where Ajax will still work on the form?

Oh my, this change destroyed one of the most important features of taxonomy manager: quickly clicking and navigating between terms. Created a citical bug report at #3505219: Restore term clicking AJAX functionality, will post a fix shortly for all AJAX fans.

This issue has been committed before. Keeping it closed.
Cheers @klausi for opening #3505219: Restore term clicking AJAX functionality. Please move all future commits there.

One of the reasons Ajax was removed is that it stopped working with media libraries and other Ajax dependent modules a while back.

We now have a new approach for the AJAX functionality in #3505219: Restore term clicking AJAX functionality that works with multi-value fields and media libraries.

Please test the merge request there. If there are no objections I'm planning to merge it in a couple of days.

Thanks @torfj for working on it so hard!