With Meta Tags: Context enabled,

Edit an existing tag in the module config (Configuration > Meta Tags > By Path)

If you click cancel, the tag will be deleted and no longer show in the list of tags.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DamienMcKenna’s picture

Priority: Normal » Major
Issue tags: +D7 stable release blocker

Yikes.

DamienMcKenna’s picture

Component: User interface » Context
cleverhoods’s picture

Checked the code.
If you comment out the L:151 from the metatag_context.admin.inc it will work.

function metatag_context_config_edit_form_cancel_submit($form, &$form_state) {
  //context_delete($form_state['metatag_context']['context']);
  $form_state['redirect'] = 'admin/config/search/metatags/context';
}

I do not really understand why would someone think the "Cancel" is equal with the "Delete".

cleverhoods’s picture

Status: Active » Needs review
DamienMcKenna’s picture

FileSize
3.36 KB

Thank you for noticing that, here is the fix as a patch.

DamienMcKenna’s picture

FileSize
3.77 KB

A better way of handling this - replace the Cancel button with the same Cancel link used on other forms.

DamienMcKenna’s picture

Status: Needs review » Fixed

Committed.

DamienMcKenna’s picture

Status: Fixed » Closed (fixed)

Now that v7.x-1.0-beta7 is out, I'm closing this in the interest of keeping the issue queue clean.