Hello all,
The code:
if (!$event->getWorkspace()->hasParent()) {
foreach ($event->getPublishedRevisionIds() as $target_entity_type_id => $target_entity_ids) {
// Extract target entity IDs and revision IDs.
$target_entity_ids_list = array_keys($target_entity_ids);
$target_entity_revision_ids = array_keys($target_entity_ids);
$this->deleteAssociations(NULL, $target_entity_type_id, $target_entity_ids_list, $target_entity_revision_ids);
}
}
which was committed to the core, there is a bug in this line:
$target_entity_ids_list = array_keys($target_entity_ids);
The $target_entity_ids variable is an array where keys - revision IDs and values - entity IDs. So instead of:
$target_entity_ids_list = array_keys($target_entity_ids);
should be:
$target_entity_ids_list = array_values($target_entity_ids);
to get entity IDs instead of revisions.
Hello all here,
I had the same bug and fixed it in the way of adding additional cleanup on publication to Live workspace.
See details in attached patch file.
Hello all here,
Here is implementation of Batch usage on publishing (to Live workspace).
+
Thank you for help!
--
Best regards,
Anton
Hello all here,
Thank you for your help!
--
Best regards,
Anton
Hello all here,
Thank you for your help! Merged.
--
Best regards,
Anton
Antonnavi → made their first commit to this issue’s fork.
Hello all here,
Thank you for your help! Merged.
--
Best regards,
Anton
Hello all here!
Thank you for help!
Hello aharown07 → ,
Flag counter can be added in the template, as described by Anssi ( anderz → ) above.
--
Best regards,
Anton
Hello all here,
Thank you for your help! Merged.
--
Best regards,
Anton
Antonnavi → made their first commit to this issue’s fork.
Hello all here,
Thank you for your help! Merged.
--
Best regards,
Anton
Thank you for help! Fixed.
Hello all here,
Thank you for your help!
Hello all here,
Thank you for your help!
Hello all here,
Thank you for your help!
Hello all here,
Thank you for your help!
Hello all here,
Thank you for your help.
Changes were committed to the development branch.
Hello Daniel ( dsp1 → ) !
If attached patch solves your issue - it means you just need to set up user entity view mode (disable display of Email verified and Email verification date fields). And your problem will be solved without this patch.
--
Best regards,
Anton
Hello Gorka G. (gorkagr) → ,
Thank you for your help! Merged.
Antonnavi → made their first commit to this issue’s fork.
Hello all!
Thank you for help! Changes were committed to the development branch.
Reviewed & tested. Was added to the new release.
@tim.plunkett or @joachim is it possible to add changes:
diff -u b/core/modules/layout_builder/src/Section.php b/core/modules/layout_builder/src/Section.php
--- b/core/modules/layout_builder/src/Section.php
+++ b/core/modules/layout_builder/src/Section.php
@@ -16,7 +16,7 @@
* @see \Drupal\Core\Layout\LayoutDefinition
* @see \Drupal\layout_builder\SectionComponent
*/
-class Section implements ThirdPartySettingsInterface {
+class Section implements ThirdPartySettingsInterface, \JsonSerializable {
/**
* The layout plugin ID.
@@ -441,2 +441,11 @@
+ /**
+ * Returns a representation of the section for use in JSON serialization.
+ *
+ * @return array
+ * Representation of the section.
+ */
+ public function jsonSerialize() {
+ return $this->toArray();
+ }
+
}
to the issue branch (to be able to export of sections through the jsonapi module on request like /jsonapi/page_variant/page_variant).
UPD: Added, found how to do it by myself. Sorry for bothering.
Patch for core 9.3.x (posted in #178) was updated:
Added exposing of sections on request (through jsonapi module /jsonapi/page_variant/page_variant) of page variant which was built as a "Layout Builder". See more details here:
https://www.drupal.org/project/page_manager/issues/3170526 →