- Issue created by @attilatilman
- š§šŖBelgium wim leers Ghent š§šŖšŖšŗ
See the diagram ā in āØ The status badge should indicate if there are changes to the page Active .
isNew
is computed by this heuristic:private function contentEntityIsConsideredNew(string $current_entity_label, EntityTypeInterface $entity_type): bool { return $current_entity_label == ApiContentControllers::defaultTitle($entity_type); }
+
public static function defaultTitle(EntityTypeInterface $entity_type): TranslatableMarkup { return new TranslatableMarkup('Untitled @singular_entity_type_label', ['@singular_entity_type_label' => $entity_type->getSingularLabel()]); }
Together, they mean that this code from this issue's MR:
// Update title and status. $duplicate->set('title', $duplicate->label() . ' (Copy)'); $duplicate->set('status', FALSE);
ā¦ will never result in
isNew: true
for duplicates.That heuristic was introduced in āØ The status badge should indicate if there are changes to the page Active 's BE MR. This MR will have to refine that heuristic.
- First commit to issue fork.
- š§šŖBelgium wim leers Ghent š§šŖšŖšŗ
Given @larowlan's work on š Include the preview HTML in the layout controller payload Active , I think he's probably best positioned to review this.