Brisbane, Australia
Account created on 15 December 2009, over 15 years ago
#

Merge Requests

More

Recent comments

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks, everyone. Committed. 🥂

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus created an issue.

🇦🇺Australia VladimirAus Brisbane, Australia

I got similar error when trying to "Replicate" the submission.

Symfony\Component\Routing\Exception\MissingMandatoryParametersException: Some mandatory parameters are missing ("webform") to generate a URL for route "entity.webform_submission.canonical". in Drupal\Core\Routing\UrlGenerator->doGenerate() (line 180 of core/lib/Drupal/Core/Routing/UrlGenerator.php).

Error is happening on tab of the submission

bit not in the the submission list

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks @kul.pratap.
Seems like you brought many other changes. Please, only commit changes related to the issue, not 1000 files.

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks @nginex. Production data is too complex to export.
Let me setup clear instance and try to replicate it.
From what I investigated, entities that are referenced in text fields are being exported as well (both taxonomies and nodes).

🇦🇺Australia VladimirAus Brisbane, Australia

Nope, getting

The website encountered an unexpected error. Try again later.

Error: Xdebug has detected a possible infinite loop, and aborted your script with a stack depth of '512' frames in Drupal\Core\TypedData\DataDefinition->getDataType() (line 54 of core/lib/Drupal/Core/TypedData/DataDefinition.php).
🇦🇺Australia VladimirAus Brisbane, Australia

After further investigation I found versions 1.4.7+ are unusable for complex websites with a lot of references.

The issues discovered:

  • Even if taxonomy terms are UNselected in config, the module still exports taxonomies
  • Taxonomies go into infinite loop, because of $this->exporter->doExportToArray($entity->get('parent')->entity) in Drupal\single_content_sync\Plugin\SingleContentSyncBaseFieldsProcessor\TaxonomyTerm::exportBaseValues()
  • Instead of single content it is now exports all the content linked with no mechanism to stop it so it can result in hundreds of nodes instead of one.
🇦🇺Australia VladimirAus Brisbane, Australia

Digging further, the module collects same entity multiple times so looks like it is cyclic until it runs out of memory or times out.

 array:2 [▼
  0 => "taxonomy_term"
  1 => "1336"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "439"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "1336"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "439"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "1336"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "439"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "1336"
]
 array:2 [▼
  0 => "taxonomy_term"
  1 => "439"
]
🇦🇺Australia VladimirAus Brisbane, Australia

Pushed to 5.0.x
needs review and testing.

🇦🇺Australia VladimirAus Brisbane, Australia

All done. Thanks everyone. 🍻

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks for reporting @paulmartin84.
Committing now.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch 3428424-automated-drupal-11 to hidden.

🇦🇺Australia VladimirAus Brisbane, Australia

Reviewed. Minor fixes added.

🇦🇺Australia VladimirAus Brisbane, Australia

Documentation reviewed. One path issue found and fixed.

🇦🇺Australia VladimirAus Brisbane, Australia

Created MR#45 that works with the latest version.
It does work for me, however if the button if not added to CKEditor, it kills the whole page still.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch ckeditor5-alt to hidden.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch 3274028-ckeditor-5-compatibility to hidden.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch ckeditor5 to active.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch ckeditor5 to hidden.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

Added merge request.
+1 for merge

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch 3435462-drupal-11 to active.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch 3435462-drupal-11 to hidden.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch 3435462-drupal-11 to hidden.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

Moved D7 down and will review existing guides.

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks @klemendev for details and testing.
Anyone else having similar issues?

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

Refactored help() functions so they have output defined and use only one return statement.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

Cheers @liam morland
Test needs to be reverted as they were updated.

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks for opening the issue.

  • Can you clarify other code-quality issues?
  • I can not locate a patch to review
🇦🇺Australia VladimirAus Brisbane, Australia

I would recommend a solution similar to views where we can toggle ajax / not ajax in config.

🇦🇺Australia VladimirAus Brisbane, Australia

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

🇦🇺Australia VladimirAus Brisbane, Australia

Tested on 10.4. Ajax functionality is back, but styling for media library and clicking media library is broken.

One of the reasons Ajax was removed in 🐛 AJAX error when editing term content Postponed: needs info is that it stopped working with media libraries and other Ajax dependent module.

🇦🇺Australia VladimirAus Brisbane, Australia

This issue was committed. Keep it close.
Cheers @klausi for opening 🐛 Restore term clicking AJAX functionality Active .

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus made their first commit to this issue’s fork.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.
Closing as outdated.

Production build 0.71.5 2024