After upgrading to 4.7 the search button don't work anymore when you use the Blix theme.
(sorry to fill this bug here but the category blix don't have been created in the bug tracking system).

Comments

thierry_gd’s picture

I found from where come the problem : it come from the modification in PHPtemplate in 4.7.

To make the button "search" work with blix theme in version 4.7, you have to modify the file page.tpl.php.

thierry_gd’s picture

 

I found from where come the problem : it come from the modification in PHPtemplate in 4.7.

To make the button "search" work with blix theme in version 4.7, you have to modify the file page.tpl.php.

Replace :

<form action="<?php print $search_url ?>" method="post" id="searchform">

<fieldset>

<input type="text" value="" name="keys" id="s" />

<input type="submit" value="Go!" id="searchbutton" />

</fieldset>

</form>

by :

<?php print $search_box ?>

I don't have any CVS access, so I hope someone will patch the 4.7 version.

Don't apply to the 4.6 version.

thierry_gd’s picture

Status: Active » Needs review

Had forgotten to change the status !!!

dries’s picture

Status: Needs review » Active

Not a patch.

Steve Dondley’s picture

blix theme? Is this an official theme release with 4.7? If not, this shouldn't be applied.

greggles’s picture

Version: 4.7.0-beta2 » 4.6.5
Priority: Critical » Normal

Blix theme is not in the Project drop down - so we can't reassign this to that theme.

I contacted adrian (blix maintainer http://drupal.org/project/blix ) via his "user contact" function yesterday asking to add it to the issues - no response yet.

Either way, this isn't "critical".

greggles’s picture

Version: 4.6.5 » 4.7.0-beta2

Sorry - didn't mean to move the "version".

Steve Dondley’s picture

Why should Drupal developers be responsible for whether or not someone's theme works with the core?

Steve Dondley’s picture

Nevermind last comment. I thought you were asking a question like "This isn't critical?"

greggles’s picture

Project: Drupal core » Drupal.org site moderators
Component: theme system » web site

This needs to be assigned to Blix theme, but that theme does not have an entry.

Assigning to Drupal.org maintenance - can you please create the issues entries for the "Blix" theme?

thierry_gd’s picture

Status: Active » Reviewed & tested by the community
StatusFileSize
new3.65 KB

Created and tested the patch

greggles’s picture

Project: Drupal.org site moderators » Blix
Version: 4.7.0-beta2 » master
Component: web site » Code

Woohoo - blix theme is listed in dropdown so this can be reassigned.

I did not test the patch, but at least there is one now.

labinformatique’s picture

Assigned: Unassigned » labinformatique
Status: Reviewed & tested by the community » Fixed
StatusFileSize
new2.39 KB

Bullshit ! Bullshit ! Bullshit !!!

Replacing the Blix search form by print $search_box puts an ugly search form in the template !

The issue is to replace

<form action="<?php print $search_url ?>" method="post" id="searchform">
    <fieldset>
        <input type="text" value="" name="keys" id="s" />
        <input type="submit" value="Go!" id="searchbutton" />
    </fieldset>
</form>

by

<form action="?q=search/node" method="post" id="searchform"> 
    <fieldset>
       <input type="text" value="" name="edit[search_theme_form_keys]" id="s" /> 
       <input type="submit" value="Go!" name="op" id="searchbutton" />
       <input type="hidden" name="edit[form_id]" id="edit-form_id" value="search_theme_form"  />
    </fieldset>
</form>

in page.tpl.php

Here was the different problems...

- action missing (<? print $serch_url; ?> doesn't work anymore)
- the field name for keywords has changed (edit[search_theme_form_keys] instead of edit[keys])
- input type="hidden" name="edit[form_id]" id="edit-form_id" value="search_theme_form" /> missing

Here is the new file page.tpl.php

thierry_gd’s picture

There 's no need to be rude, we are just human and sometimes make mistakes. Open-source need collaborative effort not agression

"Replacing the Blix search form by ... puts an ugly search form in the template !" ==> I agree with that, but the text I highlited is only a part of the correction I have planned. The css file has also to be updated for having a good rendition. I am still working on it as there is many improvment to be done on the css file as there is many problem with modules wich don't work correctly with blix themes as it is now.
I should upload a final version soon (as soon I get a little free time to validate it).

Anonymous’s picture

Status: Fixed » Closed (fixed)
thierry_gd’s picture

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

Someone (anonymous) closed it by mistake

thierry_gd’s picture

StatusFileSize
new42.62 KB

Update : modified a few extra stuff (style applied to all buttons not only the serach form ones, fixed some placement problem, ...)

thierry_gd’s picture

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

Closed ==> update to drupal 5