I would like to add support for pathauto. In order to do so, it must be excluded from HTML purification.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

pdrake’s picture

Status: Active » Needs review
FileSize
944 bytes
tloudon’s picture

Issue summary: View changes

Hi pdrake,

What's the use-case here? You can already set the path to be whatever you want in the CSV w/ the flatfish gem. I don't see any benefit to adding pathauto, but I'm not opposed to adding it in if there's a common use-case and it doesn't force pathauto or break existing functionality.

Thanks!

tloudon’s picture

Status: Needs review » Postponed (maintainer needs more info)
pdrake’s picture

Status: Postponed (maintainer needs more info) » Needs review

The use-case would be when pathauto is configured to generate paths that are dependent on a token whose value is not available during creation of the CSV. The patch does not force pathauto in any way - it just allows it to be used if desired.

tloudon’s picture

Assigned: Unassigned » tloudon
Status: Needs review » Postponed (maintainer needs more info)

Hi pdrake,

Okay, that's fine.

Can you please attach a sample CSV (just a header and one line) and a sample table dump (just one row)? Then I can document the feature so others can take advantage of it as well.

Thanks!

pdrake’s picture

Unfortunately, I'm no longer involved in the project that was using this and no longer have access to any of the code or data for that project.