In search_config_permission:


/**
 * Implements hook_permission().
 */
function search_config_permission() {
  $permissions = array(
    'admin node search exlusions' => array(

exlusions => exclusions

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

djdevin created an issue. See original summary.

joyceg’s picture

Assigned: Unassigned » joyceg
Status: Needs review » Active

I am taking up this issue.

joyceg’s picture

joyceg’s picture

Status: Active » Needs review
Alan D.’s picture

Status: Needs review » Needs work

needs update scripts. et al, so not just a trivial typo fix. maybe defer till D8?

naveenvalecha’s picture

great start joy, we also need an hook_update_n, you can read about it from here https://api.drupal.org/api/drupal/modules%21system%21system.api.php/func...

Alan D.’s picture

Category: Bug report » Task

Switching to a task as nothing is actually broken here. And probably happier to defer this to the 8.x branch.

naveenvalecha’s picture

naveenvalecha’s picture

Status: Needs work » Needs review
FileSize
1.39 KB
541 bytes

Here we go.Added hook_update.

naveenvalecha’s picture

Assigned: joyceg » Unassigned

Need review on it.

naveenvalecha’s picture

@Alan D,
Would you provide your thoughts ? are you happy with the patch so that we can commit it to 7.x

Alan D.’s picture

Really just background noise for users, but no issues if pushed.

How does this affect the migration of the permissions from D7 to D8?

naveenvalecha’s picture

How does this affect the migration of the permissions from D7 to D8?

It will not affect b/c we will use the updated permissions b/c as per our plan we will create another stable release soon before drupal 8 stable release.

Really just background noise for users, but no issues if pushed.

is it RTBC ?

Naveen

naveenvalecha’s picture

Status: Needs review » Fixed

Alright tested on local and all looks good.
Committing and pushed to 7.x-1.x

  • naveenvalecha committed ed48da4 on 7.x-1.x
    Issue #2618222 by naveenvalecha, joyceg, Alan D.: permission spelled...

Status: Fixed » Closed (fixed)

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