Port the issue #1733762 to Drupal 8

https://www.drupal.org/node/1733762

CommentFileSizeAuthor
#8 remove_search_module-2709973-8.patch4.04 KBanish.a
#5 edit_remove_search-2709973-1.patch4.06 KBfathimathsajila
#3 edit_remove_search-2709973-1.patch372 bytesfathimathsajila

Comments

anish.a created an issue. See original summary.

fathimathsajila’s picture

fathimathsajila commented
Assigned: Unassigned » fathimathsajila
fathimathsajila’s picture

fathimathsajila commented
StatusFileSize
new edit_remove_search-2709973-1.patch372 bytes

Removed search module dependency

anish.a’s picture

anish.a commented

Just removing dependency is not enough, I think.

fathimathsajila’s picture

fathimathsajila commented
StatusFileSize
new edit_remove_search-2709973-1.patch4.06 KB

Modified the code in 'search404_page' to remove the search module dependency

fathimathsajila’s picture

fathimathsajila commented
Status: Needs work » Needs review
anish_zyxware’s picture

anish_zyxware commented
Issue tags: +Needs manual testing
anish.a’s picture

anish.a commented
Status: Needs review » Reviewed & tested by the community
StatusFileSize
new remove_search_module-2709973-8.patch4.04 KB

The patch needed a reroll after a commit that modified search404.info.yml

  • anish.a committed cd06676 on 8.x-1.x 
    Issue #2709973 by fathima.v, anish.a: Remove search module dependency
anish_zyxware’s picture

anish_zyxware commented
Status: Reviewed & tested by the community » Fixed

Merged. Thanks fathima.v for the patch.

Status: Fixed » Closed (fixed)

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