Add tests for fieldable content entities with non-integer IDs

Created on 30 April 2014, almost 11 years ago
Updated 15 March 2025, 18 days ago

Problem/Motivation

Per #2236879: Broken condition in ContentEntityDatabaseStorage β†’ the support for non-integer IDs in ContentEntityDatabaseStorage is not accidental but a supported feature.

It has no test coverage.

Proposed resolution

Add test coverage:

This could probably be a DrupalUnitTestBase. We should create a new entity type, add some fields to id and perform some CRUD operations with an entity of that type.

Either in this issue or in a follow-up we should fix Entity Reference to work with this as well. It hardcodes integer IDs currently, if I'm not mistaken.

Remaining tasks

User interface changes

API changes

πŸ“Œ Task
Status

Postponed: needs info

Version

11.0 πŸ”₯

Component

entity system

Created by

πŸ‡©πŸ‡ͺGermany tstoeckler Essen, Germany

Live updates comments and jobs are added and updated live.
  • stale-issue-cleanup

    To track issues in the developing policy for closing stale issues, [Policy, no patch] closing older issues

Sign in to follow issues

Comments & Activities

Not all content is available!

It's likely this issue predates Contrib.social: some issue and comment data are missing.

  • πŸ‡ΊπŸ‡ΈUnited States smustgrave

    Thank you for creating this issue to improve Drupal.

    We are working to decide if this task is still relevant to a currently supported version of Drupal. There hasn't been any discussion here for over 8 years which suggests that this has either been implemented or is no longer relevant. Your thoughts on this will allow a decision to be made.

    Since we need more information to move forward with this issue, the status is now Postponed (maintainer needs more info). If we don't receive additional information to help with the issue, it may be closed after three months.

    Thanks!

  • πŸ‡¨πŸ‡­Switzerland berdir Switzerland

    We have several test entity types with string ids now, there are likely gaps in the test coverage for them, but I don't think still having this issue open will help with closing those, so lets close it.

Production build 0.71.5 2024