Results 1 to 10 of 205

Hybrid View

  1. #1
    Join Date
    Jan 2014
    Location
    nc
    Posts
    5
    Plugin Contributions
    0

    Default Re: Ceon Back In Stock Notifications 3.0.0

    I figured it out!

    I would have never guessed it but developer left out basic form tag. I guess zen-cart is just throwing a default one in making it work with no seo. Turn on SEO and there is no url and SEO puts in page that its on.

    Added this line to tpl_product_info_display.php and bingo it works great. now it has the correct url to submit form to. Now just have to add in to all the other templates.

    echo '<form name="back_in_stock_notification" action="http://www.mysite.com/index.php?main_page=back_in_stock_notification_subscribe&products_id=' .$_GET['products_id']. '" method="POST">';

  2. #2
    Join Date
    Jan 2014
    Location
    nc
    Posts
    5
    Plugin Contributions
    0

    Default Re: Ceon Back In Stock Notifications 3.0.0

    This is causing mysql error in account_back_in_stock_notifications.html coming from account_back_in_stock_notifications/header_php.php most likely because of the "';"; that is not allowed with my version of mysql anyway. there are two places in the script that have this coding error. but once i fix this i only get a blank page.


    $subscribed_notification_lists_query = "
    SELECT
    bisns.id, bisns.product_id, pd.products_name, bisns.date_subscribed
    FROM
    " . TABLE_BACK_IN_STOCK_NOTIFICATION_SUBSCRIPTIONS . " bisns
    LEFT JOIN
    " . TABLE_PRODUCTS_DESCRIPTION . " pd
    ON
    bisns.product_id = pd.products_id
    LEFT JOIN
    " . TABLE_CUSTOMERS . " c
    ON
    c.customers_id = bisns.customer_id
    WHERE
    (bisns.customer_id = '" . (int) $customer_id . "'
    OR
    c.customers_email_address = bisns.email_address)
    AND
    pd.language_id = '" . (int)$_SESSION['languages_id'] . "';";

  3. #3
    Join Date
    Apr 2007
    Location
    Ontario, Canada
    Posts
    1,732
    Plugin Contributions
    27

    Default Re: Ceon Back In Stock Notifications 3.0.0

    what are 'fixing' that query with?
    Twitch.
    https://www.twitchtoo.com Do you work for free? Please donate.
    June 7.2026 - NOW AVAILABLE - Twitch Base8 - Obsidian

  4. #4
    Join Date
    Jan 2007
    Location
    Los Angeles, California, United States
    Posts
    10,021
    Plugin Contributions
    32

    Default Re: Ceon Back In Stock Notifications 3.0.0

    Quote Originally Posted by speedmaxtor View Post
    I figured it out!

    I would have never guessed it but developer left out basic form tag. I guess zen-cart is just throwing a default one in making it work with no seo. Turn on SEO and there is no url and SEO puts in page that its on.

    Added this line to tpl_product_info_display.php and bingo it works great. now it has the correct url to submit form to. Now just have to add in to all the other templates.

    echo '<form name="back_in_stock_notification" action="http://www.mysite.com/index.php?main_page=back_in_stock_notification_subscribe&products_id=' .$_GET['products_id']. '" method="POST">';
    I would think that the correct way to do this is to use the "zen_draw_form()" function.. but someone can correct me if I am wrong..
    My Site - Zen Cart & WordPress integration specialist
    I don't answer support questions via PM. Post add-on support questions in the support thread. The question & the answer will benefit others with similar issues.

  5. #5
    Join Date
    Sep 2009
    Location
    Stuart, FL
    Posts
    13,942
    Plugin Contributions
    96

    Default Re: Ceon Back In Stock Notifications 3.0.0

    Quote Originally Posted by DivaVocals View Post
    I would think that the correct way to do this is to use the "zen_draw_form()" function.. but someone can correct me if I am wrong..
    Using zen_draw_form, as DivaVocals indicated, will also automatically include the hidden input for the session variable ... without which forms using the 'post' method will fail on Zen Cart versions later than v1.5.0.

  6. #6
    Join Date
    Aug 2007
    Location
    Gijón, Asturias, Spain
    Posts
    2,835
    Plugin Contributions
    31

    Default Re: Ceon Back In Stock Notifications 3.0.0

    I would have never guessed it but developer left out basic form tag.
    Since I have been using this plugin for ages with no issues, as is usually the case with Conor's code...this line finally got me annoyed enough to look in the plugin code for tpl_product_info_display.php.

    I find these snippets which build the form:
    PHP Code:
      $form_start_tag zen_draw_form('back_in_stock_notification',    zen_href_link(FILENAME_BACK_IN_STOCK_NOTIFICATION_SUBSCRIBEzen_get_all_get_params(),    $request_type), 'POST');    $back_in_stock_notification_form->setVariable('back_in_stock_notification_form_start_tag',    $form_start_tag); 
    followed by
    PHP Code:
      print $back_in_stock_notification_form->getXHTMLSource(); 
    So if you are having to manually add in the form code/reinvent this particular wheel, that means the plugin code is not being output as per design, so you need to investigate the conditions:
    PHP Code:
    if (isset($back_in_stock_notification_build_form) && $back_in_stock_notification_build_form) { 
    and find out why.
    Steve
    github.com/torvista: BackupMySQL, Structured Data, Multiple Copy-Move-Delete, Google reCaptcha, Image Checker, Spanish Language Pack and more...

  7. #7
    Join Date
    May 2008
    Posts
    402
    Plugin Contributions
    0

    Default Re: Ceon Back In Stock Notifications 3.0.0

    Does anyone know how the subscriber's IP address/email address and info can be attached to the bottom on the admin emails? For example the "For Office Use Only" part. Is there something that can be added to the email code or something that can add this stuff?

  8. #8
    Join Date
    Aug 2007
    Location
    Gijón, Asturias, Spain
    Posts
    2,835
    Plugin Contributions
    31

    Default Re: Ceon Back In Stock Notifications 3.2.2

    I have updated this module to allow for sending emails in the language that corresponds to the clients subscription language.
    It's also tested on PHP7 with mysqli using Zen Cart 1.5.5a, it should drop straight in.

    I will be uploading it shortly but would welcome some feedback/testing first:
    https://dl.dropboxusercontent.com/u/...3.2.2alpha.zip
    Steve
    github.com/torvista: BackupMySQL, Structured Data, Multiple Copy-Move-Delete, Google reCaptcha, Image Checker, Spanish Language Pack and more...

  9. #9
    Join Date
    Nov 2007
    Location
    Sunny Coast, Australia
    Posts
    3,378
    Plugin Contributions
    9

    Default Re: Ceon Back In Stock Notifications 3.2.2

    Quote Originally Posted by torvista View Post
    I have updated this module to allow for sending emails in the language that corresponds to the clients subscription language.
    It's also tested on PHP7 with mysqli using Zen Cart 1.5.5a, it should drop straight in.

    I will be uploading it shortly but would welcome some feedback/testing first:
    https://dl.dropboxusercontent.com/u/...3.2.2alpha.zip
    - BTW, I have version 3.2.0 working without a single glitch in a store which I just upgraded to ZC 1.5.5a (PHP 5.6.20)

 

 

Similar Threads

  1. Replies: 146
    Last Post: 13 Dec 2020, 09:52 AM
  2. Back In Stock Notifications
    By conor in forum All Other Contributions/Addons
    Replies: 509
    Last Post: 11 May 2016, 03:02 PM
  3. Replies: 4
    Last Post: 14 Feb 2013, 09:33 PM
  4. v139h Back In Stock Notifications only notify customers ONE time?
    By SRQHoyas in forum General Questions
    Replies: 0
    Last Post: 9 Jun 2012, 01:58 PM
  5. Problems with Back In Stock Notifications addon
    By dhanesh in forum All Other Contributions/Addons
    Replies: 0
    Last Post: 21 Aug 2008, 02:26 PM

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •  
disjunctive-egg