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

Merge Requests

More

Recent comments

🇦🇺Australia VladimirAus Brisbane, Australia

Two new merge requests and no comments?
Can you elaborate @pemson18?

🇦🇺Australia VladimirAus Brisbane, Australia

We do not need to use use function... for global functions.
Was that suggested by AI?

🇦🇺Australia VladimirAus Brisbane, Australia

Closing as outdated. Drupal version no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Closing as outdated. Drupal version no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Closing as outdated. Drupal version no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Closing as outdated. Drupal version no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 6 is no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 6 is no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is no longer supported.

🇦🇺Australia VladimirAus Brisbane, Australia

Committed and released.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus created an issue.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is outta here 👋

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is outta here 👋

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is outta here 👋

🇦🇺Australia VladimirAus Brisbane, Australia
🇦🇺Australia VladimirAus Brisbane, Australia

Doneski

🇦🇺Australia VladimirAus Brisbane, Australia

Replaced core with core_version_requirement.

🇦🇺Australia VladimirAus Brisbane, Australia

Removed Drupal 8 reference.

🇦🇺Australia VladimirAus Brisbane, Australia

Send email to maintainer.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus created an issue.

🇦🇺Australia VladimirAus Brisbane, Australia

Drupal 7 is deprecated

🇦🇺Australia VladimirAus Brisbane, Australia

Thank you. Committed.

🇦🇺Australia VladimirAus Brisbane, Australia

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

🇦🇺Australia VladimirAus Brisbane, Australia

Thank you @chri5tia.
Can you add MR and remove commented lines if they are nor required?

🇦🇺Australia VladimirAus Brisbane, Australia

@ggh this is not Drupal 12 issue.
You need to read your errors.

🇦🇺Australia VladimirAus Brisbane, Australia

Yeah, I'm still happy to help.
Back from my Drupal holidays

🇦🇺Australia VladimirAus Brisbane, Australia

So many vultures...

🇦🇺Australia VladimirAus Brisbane, Australia

Another +1. Crashed after the update. Patched fixed the crash.

🇦🇺Australia VladimirAus Brisbane, Australia

Thanks @jurgenhaas and @jsacksick.

🇦🇺Australia VladimirAus Brisbane, Australia

vladimiraus changed the visibility of the branch 3077612-missing-handling-of to hidden.

🇦🇺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

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

vladimiraus changed the visibility of the branch 2.x 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.

Production build 0.71.5 2024