Hello:

In the window: test.dev/admin/config/domain/add

The name machine doesn't change to write the server name.

the error appears when filling in the second domain.

Drupal\Core\Entity\EntityStorageException: 'domain' entity with ID 'test_dev' already exists. en Drupal\Core\Entity\EntityStorageBase->doPreSave() (linea 425 de /var/www/test.dev/core/lib/Drupal/Core/Entity/EntityStorageBase.php).

CommentFileSizeAuthor
#6 2927568-domain-add-6.patch4.5 KBagentrickard
#5 2927568-domain-add.patch1.21 KBagentrickard
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Chismen created an issue. See original summary.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Active » Postponed (maintainer needs more info)

This report is not replicable as currently written. Please provide more detail about the steps taken -- data entered, number of domains created, hostnames entered. Note that the machine_name (which causes the error) is derived from the hostname, which must be unique.

This may have been fixed in #2912150: Relation not updated with machine name changes as well.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Postponed (maintainer needs more info) » Active

Hm, I see the "machine name does not change" piece now. I wonder why?

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented

Must be a change in Drupal 8.4.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Active » Needs review
FileSize
2927568-domain-add.patch1.21 KB

Here's a patch against dev. Not sure if it applies cleanly to alpha9.

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
FileSize
2927568-domain-add-6.patch4.5 KB

Revised patch with passing tests. Combining into https://github.com/agentrickard/domain/pull/385

mitrpaka’s picture

mitrpaka CreditAttribution: mitrpaka as a volunteer commented
Status: Needs review » Reviewed & tested by the community

Was having the very same issue with fresh 8.4.2 and domain (latest dev) installation. Patch did resolve machine name issue while adding new domain record.

I guess this is partly due to regression from #2912150

agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented

Looks like it. I'm working on tests now. Thanls!

  • agentrickard committed 86ac29f on 8.x-1.x 
    Fix for issue #2927568 reported by Chismen: domain add error
agentrickard’s picture

agentrickard
he/him
Primary language English
Location Georgia (US)
CreditAttribution: agentrickard at Palantir.net commented
Status: Reviewed & tested by the community » Fixed
Chismen’s picture

Chismen CreditAttribution: Chismen as a volunteer commented

thanks, works fine !!

Status: Fixed » Closed (fixed)

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