$ drush pmu skinr_ui
The following extensions will be uninstalled: skinr_ui
Do you really want to continue? (y/n): y
skinr_ui was successfully uninstalled. [ok]
PHP Fatal error: Class Drupal\search404\Form\Search404Settings contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Drupal\Core\Form\ConfigFormBase::getEditableConfigNames) in /usr/share/nginx/html/drupal9/modules/contrib/search404/src/Form/Search404Settings.php on line 187
Drush command terminated abnormally due to an unrecoverable error. [error]
Error: Class Drupal\search404\Form\Search404Settings contains 1 abstract method and must
therefore be declared abstract or implement the remaining methods
(Drupal\Core\Form\ConfigFormBase::getEditableConfigNames) in
/usr/share/nginx/html/drupal9/modules/contrib/search404/src/Form/Search404Settings.php, line
187

CommentFileSizeAuthor
#3 search404-fix-error-2572329-1-D8.patch489 bytesanish.a

Comments

quindio created an issue. See original summary.

kingfisher64’s picture

Yep I'm seeing this too on update.php page.

Fatal error: Class Drupal\search404\Form\Search404Settings contains 1 abstract method and must therefore be declared abstract or implement the remaining methods (Drupal\Core\Form\ConfigFormBase::getEditableConfigNames) in modules/search4042/src/Form/Search404Settings.php on line 17

anish.a’s picture

Title: contains 1 abstract method and must therefore be declared » Contains 1 abstract method and must therefore be declared
Assigned: Unassigned » anish.a
StatusFileSize
new489 bytes

This patch should fix it.

Per https://www.drupal.org/node/2407153, forms that save configuration need to define which configuration can be updated.

kingfisher64’s picture

Tried the dev module again dated 4th Dec and applied the patch but still getting this error.

I'm not sure if the 2 patches, 1 here and 1 on https://www.drupal.org/node/2612440 are needed with the new latest dev?

Would it be possible to just apply these two and release another dev version to test?

anish_zyxware’s picture

Yes. Both as essential to fix the module installation. I am waiting for commit access :)

Can you please test that by applying both patches?

kingfisher64’s picture

Hi Anish,

Can confirm that with both patches applied there is no longer any install errors or WSOD's :)

Will also confirm this fixed in other thread as well.

Thank you

kingfisher64’s picture

Priority: Normal » Major
Status: Active » Reviewed & tested by the community

  • anish_zyxware committed 65934b3 on 8.x-1.x
    #2572329 : Fix abstract method must be redeclared error.
    
anish_zyxware’s picture

Status: Reviewed & tested by the community » Closed (fixed)

This issue is fixed as per #8 and is available in 8.x-1.x-dev branch

anish_zyxware’s picture