move NodeInterface::PUBLISHED and CommentInterface::PUBLISHED/NOT_PUBLISHED to EntityPublishedInterface

Open on Drupal.org →

Created on 7 May 2025, 10 days ago

Problem/Motivation

Now that we have EntityPublishedInterface, any entity type can have a 'published' field.

But the useful constants for this field's value are only on the node and comment entity interfaces.

Steps to reproduce

Proposed resolution

Move these constants to EntityPublishedInterface.

Remaining tasks

User interface changes

Introduced terminology

API changes

New constants. We need to consider what happens if contrib/custom classes that implement EntityPublishedInterface have already defined these constants.

Data model changes

Release notes snippet

✨ Feature request

Status

Active

Version

11.0 🔥

Component

entity system

Created by

🇬🇧United Kingdom joachim

Live updates comments and jobs are added and updated live.

Novice
It would make a good project for someone who is new to the Drupal contribution process. It's preferred over Newbie.

Merge Requests

!12143move NodeInterface::PUBLISHED and CommentInterface::PUBLISHED/NOT_PUBLISHED to EntityPublishedInterface
Open
Unnamed author
updated about 17 hours ago

Comments & Activities

Issue created by @joachim
Comment 1 day ago →
am_feunj
I will try to fix it to next days.
Merge request !12143Issue #3523070: move NodeInterface::PUBLISHED and... → (Open) created by Unnamed author
Comment about 17 hours ago →
am_feunj
Pipeline finished with Success
about 17 hours ago
Total: 587s
#498910
Comment about 13 hours ago →
🇬🇧United Kingdom joachim
That's a good start, but we can't just remove constants, because people will be using them.

We have to deprecate them: see https://www.drupal.org/about/core/policies/core-change-policies/how-to-d... →

Removal will come in a future version.

While we're at it, I think that the wording from node is clearer:

> * Denotes that the [node] is not published.

contrib.social Blog FAQ Discussions

Production build 0.71.5 2024