Hello.

I've found little mistake. Key 'ajax' has been defined twice in function ctools_access_ajax_edit

  $form_state = array(
    'plugin' => $plugin,
    'id' => $id,
    'test' => &$access['plugins'][$id],
    'access' => &$access,
    'contexts' => $contexts,
    'title' => t('Edit criteria'),
    'ajax' => TRUE,
    'ajax' => TRUE,
    'modal' => TRUE,
    'modal return' => TRUE,
  );

Here are two keys 'ajax' with the same value.

Comments

eugene.ilyin created an issue. See original summary.

eugene.ilyin’s picture

Status: Active » Needs review
StatusFileSize
new443 bytes

I've prepared simple patch to correct this mistake

eugene.ilyin’s picture

Status: Needs review » Reviewed & tested by the community

Could somebody check this simple issue?

rivimey’s picture

@eugene.ilyin please do not set your own issues to RTBC, especially without other comments. It is frustrating waiting but there are only so many people who are willing to review and check patches.

In this case, as you suggest, the fix is fine. Thanks for your input.

eugene.ilyin’s picture

I'm very sorry for this

please do not set your own issues to RTBC, especially without other comments.

I'll keep it in mind. Thank you for your review.

japerry’s picture

Status: Reviewed & tested by the community » Fixed

Extra key removed!

Status: Fixed » Closed (fixed)

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