FAQ Manager

[reuben]

Andrew,

Your suggested change did fix the problem. This module now works perfectly.

Thankyou very much for your help!

Also thanks to everyone who has developed, tested, and argued about this contrib.

Cheers
Reuben

[Dehaw]

Finally a Solution!

[Dehaw]

Thanks Andrew, i have updated the package and provided a mention on that very line for you.

Thanks Again

Derek

[a_berezin]

Hi Derek,
It not the correct decision. Correctly to do so:
tpl_faqs_new_default.php, tpl_faq_categories.php, new_faqs.php

  switch (true) {
    case (SHOW_NEW_FAQS_LIMIT == '0'):
      $display_limit = '';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '1'):
      $display_limit = " and date_format(p.faqs_date_added, '%Y%m') >= date_format(now(), '%Y%m')";
      break;
    case (SHOW_NEW_FAQS_LIMIT == '30'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 30';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '60'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 60';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '90'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 90';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '120'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 120';
      break;
  }

replace by

  switch (true) {
    case (SHOW_NEW_FAQS_LIMIT == '1'):
      $display_limit = " and date_format(p.faqs_date_added, '%Y%m') >= date_format(now(), '%Y%m')";
      break;
    case (SHOW_NEW_FAQS_LIMIT == '30'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 30';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '60'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 60';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '90'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 90';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '120'):
      $display_limit = ' and TO_DAYS(NOW()) - TO_DAYS(p.faqs_date_added) <= 120';
      break;
    case (SHOW_NEW_FAQS_LIMIT == '0'):
    default:
      $display_limit = '';
      break;
  }

[a_berezin]

Also would be correct to result a code in conformity with zen-cart ideology.
Remove strange open-operations directory, don't edit any zen-cart core files, and so on.
I have no time for this.

[Dehaw]

Also would be correct to result a code in conformity with zen-cart ideology.
Remove strange open-operations directory, don't edit any zen-cart core files, and so on.
I have no time for this.

Yes i agree with this too, although i also do not have time.

This is a contribution for the community if someone else wants to do it thats great, it was Dave at Open Operations that originally developed it so i would also think that it's fair to leave some form of mention to Dave on there, and since this is also the way of the GNU/GPL licence.

Derek

[Pablosan]

So is it safe to try this mod again?