Current 3.2:

* go to admin/config/development/features/bundle and create now bundle foo
* go to admin/config/development/features, click create new feature
* For name, enter text
* Click write

Experienced:
* The machine-readable name is already in use. It must be unique.
(Looks like the core text module clashes.)

Expected:
* feature foo_text is written

CommentFileSizeAuthor
#5 features-exists-2841683-5.patch800 bytesnedjo

Comments

axel.rutz created an issue. See original summary.

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Status: Active » Postponed (maintainer needs more info)

Could you provide a bit more detail about the exact steps to reproduce this error?

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Issue summary: View changes

Updated description.

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Status: Postponed (maintainer needs more info) » Active

@axel.rutz: thanks, got it now!

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Status: Active » Needs review
StatusFileSize
new features-exists-2841683-5.patch800 bytes

Haven't tested, but this might be what's needed.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Status: Needs review » Reviewed & tested by the community

Yes, that's the machine name validator.
Tested and ready to go.

Status: Reviewed & tested by the community » Needs work

The last submitted patch, 5: features-exists-2841683-5.patch, failed testing.

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Status: Needs work » Reviewed & tested by the community

Test failure is from #2833258: Test failure from core change, so setting back to previous status.

danepowell’s picture

danepowell commented

I also encountered this bug and can confirm #5 fixes it.

nedjo’s picture

nedjo
he/him/his
Primary language English
commented
Issue tags: +Needs tests

I'd like to have an accompanying test before applying this but haven't found time to write one.

mpotter’s picture

mpotter commented

Triggering new test runs.

  • mpotter committed 900e53b on 8.x-3.x authored by nedjo 
    Issue #2841683 by nedjo: Feature name validation must respect namespace
mpotter’s picture

mpotter commented
Status: Reviewed & tested by the community » Fixed

Committed to 900e53b.

Status: Fixed » Closed (fixed)

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