The code inside pathauto_form_alter() for adding all its fields to the node's path fieldset should be abstracted to work for any entity. That way we can re-use it on taxonomy term forms.

CommentFileSizeAuthor
#4 752034-pathauto-field-attach-form.patch14.76 KBdave reid
#3 752034-pathauto-field-attach-form.patch7.48 KBdave reid
#2 752034-pathauto-field-attach-form.patch7.47 KBdave reid
#1 752034-pathauto-entity-form-D7.patch6.84 KBdave reid

Comments

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Active » Needs work
StatusFileSize
new 752034-pathauto-entity-form-D7.patch6.84 KB

Here's how it would look in patch and the simplified way to add the 'Automatic URL alias' settings to an individual entity's form.

function pathauto_form_alter(&$form, &$form_state, $form_id) {
  // Process only node forms.
  if (!empty($form['#node_edit_form'])) {
    $node = $form['#node'];
    pathauto_entity_path_edit_form($form['path'], 'node', $node);
  }
}
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs work » Needs review
StatusFileSize
new 752034-pathauto-field-attach-form.patch7.47 KB

New patch using pathauto_field_attach_form() but calling it from form_alters since hook_field_attach_form() runs before any path module alters.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
StatusFileSize
new 752034-pathauto-field-attach-form.patch7.48 KB
dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Title: Make a pathauto_entity_alias_edit_form for re-use » Allow the Pathauto form altering to be re-usable
StatusFileSize
new 752034-pathauto-field-attach-form.patch14.76 KB

With tests.

dave reid’s picture

dave reid
he/him
Primary language English
Location Nebraska USA
commented
Status: Needs review » Fixed

Committed to Git: http://drupalcode.org/project/pathauto.git/commit/43319d7

Status: Fixed » Closed (fixed)

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