Problem/Motivation

Currently when we enable search404 module we can only update custom page title and custom page text for multilingual site . But if we want to give a custom search path for each language on site we are unable to do it because it overrides the path entered in textbox

Proposed resolution

We should set this variable also has multilingual so that the user has the flexibility to change the custom path as per the languages in the site . So that each language has different custom path

Remaining tasks

User interface changes

None.

API changes
None.

Data model changes

None.

Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Shreya Shetty created an issue. See original summary.

Shreya Shetty’s picture

Title: Custom search path for multilingual » Custom path search instead of a Drupal Search when a 404 occurs for multilingual site
Shreya Shetty’s picture

Here is the patch which sets the custom path variable as a is a multilingual variable.

Shreya Shetty’s picture

Status: Active » Needs review
Shreya Shetty’s picture

anish.a’s picture

Status: Needs review » Needs work

I tested this and couldn't make it work.

The variable is not saving properly. Can you check?

anish.a’s picture

On further investigation, I found that its the issue with the multilingual variables in settings page. When I set using multilingual variable editor, it works fine. But on settings page, the value is not saving properly.

Shreya Shetty’s picture

Hi anish ,
Thanks for reviewing the patch . I had moved the settings of custom patch URL under the advanced settings tab . But now i have kept it where it was before and it appears to be fine .

Shreya Shetty’s picture

Status: Needs work » Needs review
arjun_sreekumar’s picture

Assigned: Unassigned » arjun_sreekumar
Status: Needs review » Needs work
arjun_sreekumar’s picture

anish_zyxware’s picture

Assigned: arjun_sreekumar » Unassigned
Status: Needs work » Reviewed & tested by the community
FileSize
4.24 KB

Some code cleanup. Done final patch.

anish_zyxware’s picture

Status: Reviewed & tested by the community » Fixed

Merged the code.

anish_zyxware’s picture

Status: Fixed » Closed (fixed)