Steps to reproduce:

1. Create new entity (that has Pathauto and Fieldable Path configured)
2. You should see correct Fieldable Path ('/content/test')
3. Re-save entity without touching any fields.

Current behaviour: you will see new value in Fieldable Path ('/content/test-0').

I'm assuming it's related to workaround for Pathauto in FieldablePath::preSave. I simply removed it and it worked.

Comments

kalabro created an issue. See original summary.

kalabro’s picture

kalabro’s picture

Status: Active » Needs review

  • Spleshka committed 4f0e2ad on 8.x-1.x
    Issue #2874744 by kalabro, Spleshka: new pathauto alias with "-0" suffix...
spleshka’s picture

Version: 8.x-1.0-rc1 » 8.x-1.x-dev
Status: Needs review » Fixed

Hi @kalabro. Unfortunately, we can't just remove this section, because it prevents entity from having entity save during ongoing entity save operation, which potentially might lead to unpredictable issues.

I debugged the issue and turns out the fix was quite simple: for existing entities we do need to pass a real entity id to make pathauto generate correct aliases.

Btw, thanks for the report.

Status: Fixed » Closed (fixed)

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