FAQ Manager - FAQs multiple times displayed

[FredSchenk]

LS,
I just downloaded and installed the FAQ Manager Module. It worked directly with ZenCart 1.3.9h (except for the SQL-script which I had to change manually since my table structure isn't standard, almost nothing any module can do about this).

I only had one problem: Every FAQ was displayed three times.
However: We also have three languages on our site (Dutch, English and French), so this connection was quickly made.

I changed one line in

includes\templates\<myTemplate>\templates\tpl_faqs_all_default.php

Line 41/42 has become:

    $faqs_all_query_raw = "select f.*, fd.*, fcd.faq_categories_name from " . TABLE_FAQS . " f, " . TABLE_FAQS_DESCRIPTION . " fd, " . TABLE_FAQ_CATEGORIES_DESCRIPTION . " fcd, " . TABLE_FAQ_CATEGORIES . " fc " .
            "where f.faqs_status = '1' and f.faqs_id = fd.faqs_id and fd.language_id = '" . (int) $_SESSION['languages_id'] . "' and fd.language_id = fcd.language_id and f.master_faq_categories_id = fcd.faq_categories_id and f.master_faq_categories_id = fc.faq_categories_id and fc.faq_categories_status='1' ";

adding the

and fd.language_id = fcd.language_id

solved my problem.

I've already PM-ed this to the latest uploader of this module (sunnycoolboy) but wanted to share this knowledge directly with all of you too.

With kind regards,

Fred Schenk

PS: I haven't checked to see if this is the only place where the language-check should be extended.

[FredSchenk]

OK, just continuing working with the FAQ and found another case of the language multiplier. In another template:

tpl_faqs_new_default.php

I found the following at lines 62/63/64:

  $faqs_new_query_raw = "select f.*, fd.*, fcd.faq_categories_name from " . TABLE_FAQS . " f, " . TABLE_FAQS_DESCRIPTION . " fd, " . TABLE_FAQ_CATEGORIES_DESCRIPTION . " fcd, " . TABLE_FAQ_CATEGORIES . " fc " .
                        "where f.faqs_status = '1' and f.faqs_id = fd.faqs_id and fd.language_id = '" . (int)$_SESSION['languages_id'] . "' and f.master_faq_categories_id = fcd.faq_categories_id and f.master_faq_categories_id = fc.faq_categories_id and fc.faq_categories_status='1' " .
						$order_by;

and changed it to:

  $faqs_new_query_raw = "select f.*, fd.*, fcd.faq_categories_name from " . TABLE_FAQS . " f, " . TABLE_FAQS_DESCRIPTION . " fd, " . TABLE_FAQ_CATEGORIES_DESCRIPTION . " fcd, " . TABLE_FAQ_CATEGORIES . " fc " .
                        "where f.faqs_status = '1' and f.faqs_id = fd.faqs_id and fd.language_id = '" . (int)$_SESSION['languages_id'] . "' and fd.language_id = fcd.language_id and f.master_faq_categories_id = fcd.faq_categories_id and f.master_faq_categories_id = fc.faq_categories_id and fc.faq_categories_status='1' " .
						$order_by;

So here too, adding the

and fd.language_id = fcd.language_id

solved my problem...

With kind regards,

Fred Schenk

[FredSchenk]

OK, since I didn't "trust" the code I decided to search all the php-files for

$_SESSION['language 

to see if some other changes had to be made. And I found one other file where I had to add the

 and fd.language_id = fcd.language_id 

code:

\includes\templates\ScrappedLives\templates\tpl_faq_default.php (line 40)

If I'm not mistaking, I now have found all the files/queries to make this problem disappear. Too bad the queries are all over the code and not in one place...

With kind regards,

Fred Schenk