Error: Call to a member function grantPermission() on string in /Applications/MAMP/htdocs/www/modules/d8/search_config/search_config.install,
line 54

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

naveenvalecha created an issue. See original summary.

naveenvalecha’s picture

Status: Active » Needs review
FileSize
2.61 KB
naveenvalecha’s picture

Status: Needs review » Fixed

  • naveenvalecha committed e018e85 on 8.x-1.x
    Issue #2771555 by naveenvalecha: Issue while installing module
    

Status: Fixed » Needs work

The last submitted patch, 2: 2771555-2.patch, failed testing.

naveenvalecha’s picture

Status: Needs work » Fixed

back to fixed

naveenvalecha’s picture

Explanation of the above patch.

  1. +++ b/search_config.install
    @@ -38,39 +39,13 @@ $schema = [];
    -  module_load_include('module','search_config');
    -  foreach (array(AccountInterface::ANONYMOUS_ROLE, AccountInterface::AUTHENTICATED_ROLE) as $rid) {
    -    $rid->grantPermission('search all content');
    -      }
    

    user_role_grant_permissions is better here.

  2. +++ b/search_config.install
    @@ -38,39 +39,13 @@ $schema = [];
    -    $settings = search_config_node_settings();
    -    \Drupal::service('config.factory')->getEditable('search_config.settings')->save();
    

    We'll add the default settings in config/install directory

  3. +++ b/search_config.install
    @@ -38,39 +39,13 @@ $schema = [];
    -      // Remove the old 6.x variables
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_or')->save();
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_phrase')->save();
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_negative')->save();
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_category_all')->save();
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_category')->save();
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_type')->save();
    -      \Drupal::config('search_config.settings')->clear('search_config_disable_index_type')->save();
    

    Unused/Not needed code

  4. +++ b/search_config.install
    @@ -38,39 +39,13 @@ $schema = [];
    -  \Drupal::service('config.factory')->getEditable('search_config.settings')->delete();
    -  \Drupal::service('config.factory')->getEditable('search_config.node_content_settings')->delete();
    

    We don't need to explictly remove the config as its automatically gets removed on module uninstallation by config api.

Status: Fixed » Closed (fixed)

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