XB pages doesn't respect Pathauto widget

Adding a related issue here

It's possible I duplicated this issue with 🐛 Autosave publish process does not acknowledge pathauto deactivation Postponed , but not fully sure as the IS here specifies the issue is due to pathauto creating an alias when a pattern isn't defined for the entity type, while #3526130 is specifically about the publish process not acknowledging the "Generate automatic URL alias" being unchecked.

If they are in fact the same issue reported differently please note the issue summary in #3526130 already provides a 3 line solution that works, but notes there's probably a cleaner way to implement it (perhaps in a hook that specifies module: 'pathauto'

We should first address 🐛 Autosave publish process does not acknowledge pathauto deactivation Postponed before proceeding here. Once that is fixed, this issue might be completely resolved, too.

After 🐛 Autosave publish process does not acknowledge pathauto deactivation Postponed this still doesn't work,but it now looks to be an issue in \Drupal\experience_builder\AutoSave\AutoSaveManager::saveEntity - Entity::toArray doesn't have that value in it - this is because pathauto uses key-value to store that value, not an entity field. We might have to special case pathauto in what we store in $auto_save_data

Marking this active as https://www.drupal.org/project/experience_builder/issues/3525070 🐛 XB pages doesn't respect Pathauto widget Active is done.

In the original issue summary for 🐛 Autosave publish process does not acknowledge pathauto deactivation Postponed I provided a special case snippet for Pathauto that when added to ApiAutosaveController it successfully got things working as seen here. I won't assume it's the solution, but cant hurt to have a reference from something already proven to work.

We talked about this issue in our last team meeting. While we agree it's a real problem, we decided not to move forward with it right now. Instead of fixing it specifically for Pathauto, we’d like to step back and think about a more general approach that could work for other similar cases too.

Marking this as postponed until we revisit it with a broader solution in mind.

wim leers → credited mayur-sose → .

Transplanting #3526130-32: \Drupal\experience_builder\ClientDataToEntityConverter::setEntityFields doesn't work with booleans outside of boolean field → here 😊 Thanks, Mayur!