Problem/Motivation

The RokkaMetadata content entity type exposes its publication state, but does this in its own way which is subtly different to how it is usually done in Drupal.

Proposed resolution

  • Implement EntityPublishedInterface and adopt EntityPublishedTrait.
  • Preserve backwards compatibility, but issue deprecation warnings for code that uses the current API where it differs from the official API.
  • Create a 4.x branch that is fully conforming.

Issue fork rokka-3558439

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

pfrenssen created an issue. See original summary.

pfrenssen opened merge request !35

  • pfrenssen committed d18a8239 on 3.x 
    feat: #3558439 RokkaMetadata should implement EntityPublishedInterface...

pfrenssen opened merge request !36

  • pfrenssen committed d18a8239 on 4.x 
    feat: #3558439 RokkaMetadata should implement EntityPublishedInterface...

  • pfrenssen committed 73f1f00a on 4.x 
    feat: #3558439 Remove deprecated $published argument

By: @pfrenssen
pfrenssen’s picture

pfrenssen
Location Sofia
commented
Status: Active » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.