Search returning errors v1.2.x

[tamarabrown]

We have found that if you run a search from the sidebox and then from the results page run another search from the sidebox an error is returned. In the Search sidebox enter horse and search. The results page is fine. In the sidebox enter a new term and search and the error below is returned.
-----------------------------
Warning: Variable passed to each() is not an array or object in /home/clipart/public_html/includes/application_top.php on line 712

Warning: Cannot modify header information - headers already sent by (output started at /home/clipart/public_html/includes/application_top.php:712) in /home/clipart/public_html/includes/functions/functions_general.php on line 53
-----------------------------

The address of the first search is:
index.php?main_page=advanced_search_result&search_in_description=1&keyword=horse

The address of the second page is:
index.php?main_page=products_general_info&search_in_description=1&keyword=horse& sort=20a&action=multiple_products_add_product

We are running
Zen Cart 1.2.7
Database Patch Level: 1.2.7
on Linux with MySQL 4.0.25 and PHP 4.4.1
I am not running any Mods.
See www.3DClipart.net.

I do not have a solution for this problem. Suggestions are most welcome.

Tamara

[Absolute]

Just seen your problem, and that is that its not submitting the correct form on the advanced search page. Have you editted the way the page displays? Open up includes/templates/YOUR ACTIVE TEMPLATE/templates/tpl_advanced_search_default.php . DO you have </form> on the penultimate line? The last two lines should be

</form>
</div>

If you close the form on this page correctly, then the search box will then function as it should.

Absolute

[tamarabrown]

No, I haven't made any changes to these pages. I checked the source and the closing </form> is at the bottom. However, there are no <div> tags in the code.

But the error does not appear to be occurring on advanced_search_result. The problem is in the side block. It appears to redirect to the wrong page.

Tamara

[Absolute]

It doesn't matter where the error is appearing, its the fact that a form is openned for the centre column, so that you can add products, and then another form is openned for the advanced search box. Because you never close the first form, the advance search box is submitting the data to the previous form. This is why when you click search on the advanced search results page, you get multipe_add_products in the url of the page returning the error.

If you want to see what I mean for yourself, then view the advanced search page, and view the source. Then, in notepad so a search for the word 'form'. That way you can see when the first is openned, and then the second, and then only the second is closed.

Absolute

[DrByte]

The problem is in your tpl_modules_product_listing.php template file.

I can duplicate the problem on your site by clicking the Animals->Horses category, and then entering anything in the search sidebox and clicking Search.
The problem is repeatable whenever you have the product-listing page open.

Thus, the tpl_modules_product_Listing.php file must have the closing </form> tag missing from it.

In v1.2.7, the last 7 lines of the file are:

<?php
    } // show_bottom_submit_button
?>
<?php if ($show_top_submit_button == 'true' or $show_bottom_submit_button == 'true') { ?>
</form>
<?php } ?>
</table>

I suspect maybe you have some sort of addon installed or may be some switch setting in the admin area that's causing this.

At any rate, the problem doesn't exist in v1.3.x

[tamarabrown]

I am afraid that you were both wrong. But, Absolute was closer to the solution than DrByte. First, I had not edited any of the files mentioned. I have done very little editing to this store and do not have any add-on modules running.

The problem was in tpl_advanced_search_result_default.php. A closing </form> tag needed to be added at line 35.

Both tpl_advanced_search_default.php and tpl_modules_product_listing.php had the closing form tag in the original files.

Thank you for pointing me in the right direction.
Tamara

[europod]

I am getting a similar error I have tried to take the advice on board but it is not working and this error is still appearing. If the problem if with the lack of the </form> where do I add it to tpl_advanced_search_result_default.php

HERE IS THE ERROR

Warning: Variable passed to each() is not an array or object in /home/content/e/u/r/eurovisionshop/html/includes/application_top.php on line 682

Warning: Cannot modify header information - headers already sent by (output started at /home/content/e/u/r/eurovisionshop/html/includes/application_top.php:682) in /home/content/e/u/r/eurovisionshop/html/includes/functions/functions_general.php on line 52

[europod]

Still I am receiving this error and cannot seem to fix it! Any help would me much appreciated. EP