Problem/Motivation

Right now, created-type fields are exported by the default content exporter. But this really doesn't make a lot of sense; when you import default content, you probably want that content to carry the time it was imported, not the time it was created by whoever originally built that default content.

Proposed resolution

We already ignore changed fields; let's also ignore created fields.

Issue fork drupal-3557465

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

phenaproxima created an issue. See original summary.

phenaproxima opened merge request !13806

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Active » Needs review
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented

There's no need for explicit test coverage here; the removal of created from the default content fixture is sufficient, because ContentExportTest already asserts that the exported content is identical to what's in the fixture.

thejimbirch’s picture

thejimbirch commented
Status: Needs review » Reviewed & tested by the community

Thanks, moving to RTBC.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Reviewed & tested by the community » Needs work
Issue tags: +Needs change record

I can see why you might want this though... but also @phenaproxima tells me an event subscriber can add it back in. So lets add a CR and tell people how to do that.

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
commented
Status: Needs work » Reviewed & tested by the community
Issue tags: -Needs change record

Done. https://www.drupal.org/node/3557689

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Version: 11.x-dev » 11.3.x-dev
Status: Reviewed & tested by the community » Fixed

Committed and pushed 2b63f874753 to 11.x and 06d7aaadbe9 to 11.3.x. Thanks!

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.

  • alexpott committed 06d7aaad on 11.3.x 
    Issue #3557465 by phenaproxima, thejimbirch, alexpott: The default...

  • alexpott committed 2b63f874 on 11.x 
    Issue #3557465 by phenaproxima, thejimbirch, alexpott: The default...

Status: Fixed » Closed (fixed)

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