Jump to content
b00

[Contribution] Recently Viewed Products(sales optimized)

Recommended Posts

Share this post


Link to post
Share on other sites

i had the same problem images not showing on the recently_viewed.php but showing in the boxes ok

The recently_viewed.php has the p.products_image missing from the query, add that to the query and all works fine.

i'm using OSCOM CE Phoenix v1.0.5.0

 

Share this post


Link to post
Share on other sites
11 hours ago, nigeloscomm said:

i had the same problem images not showing on the recently_viewed.php but showing in the boxes ok

The recently_viewed.php has the p.products_image missing from the query, add that to the query and all works fine.

i'm using OSCOM CE Phoenix v1.0.5.0

 

You are using an outdated version which still needs the PRODUCT_LIST_IMAGE setting set to "true".

Please update to the latest version:

                Version 5.0.2 Phoenix

                Changes:        
         - removed deprecated "Product Image" configuration setting

 

Share this post


Link to post
Share on other sites

It appears that the core changes introduced in 1.0.7.3 are not compatible with this add-on.

Once I get the new 1.0.7.4 changes merged in I will see if I can update the add-on to work with these newer versions.

If anyone else happens to be looking at this same issue, please let me know if or how I can help.

Thanks, Dave

 

Share this post


Link to post
Share on other sites
3 minutes ago, grumplestiltskin said:

It appears that the core changes introduced in 1.0.7.3 are not compatible with this add-on.

Once I get the new 1.0.7.4 changes merged in I will see if I can update the add-on to work with these newer versions.

If anyone else happens to be looking at this same issue, please let me know if or how I can help.

Thanks, Dave

 

The update is on my to do list. If you get it updated, please mail me the updated package or post it here. So I can add it to the marketplace.

Thanks Rainer

Share this post


Link to post
Share on other sites
42 minutes ago, raiwa said:

The update is on my to do list. If you get it updated, please mail me the updated package or post it here. So I can add it to the marketplace.

Thanks Rainer

Will do! I'm happy to give back whatever I can....

Share this post


Link to post
Share on other sites

Uploaded update:

Version 5.1.0. Phoenix
Requires Phoenix 1.0.7.2+

Changes:        
   - updated for Phoenix 1.0.7.2+
   - updated all modules to abstract executable modules
   - update recently_viewed.php to templateable page
   - updated hook and moved to templates/


Full Package
Update Instructions included

Share this post


Link to post
Share on other sites

Language file german for

cm_i_recently_viewed.php

needs corrections

------------------------------------

in cm_pi_recently_viewed.php

MODULE_CONTENT_SC_RECENTLY_VIEWED_CONTENT_LIMIT

must be MODULE_CONTENT_PI_RECENTLY_VIEWED_CONTENT_LIMIT

-------------------------------------

I hope I did not forget anything ...

Share this post


Link to post
Share on other sites

Uploaded update with the above fixes

Version 5.1.1. Phoenix
Requires Phoenix 1.0.7.2+

Changes:        
         - fix in german language file cm_i_recently_viewed.php. Thanks to @rupruprup for the report
         - fixed error in cm_pi_recently_viewed.php. Thanks to @rupruprup for the report

Share this post


Link to post
Share on other sites

in cm_pi_recently_viewed.php

const CONFIG_KEY_BASE = 'MODULE_CONTENT_INDEX_RECENTLY_VIEWED_';

should be const CONFIG_KEY_BASE = 'MODULE_CONTENT_PI_RECENTLY_VIEWED_';

Share this post


Link to post
Share on other sites

Uploaded update with the above fixes

Version 5.1.2. Phoenix
Requires Phoenix 1.0.7.2+

 Changes:        
         - fixed error in cm_pi_recently_viewed.php. Thanks to @rupruprup for the report

 

Share this post


Link to post
Share on other sites

Uploaded update

Version 5.1.3. Phoenix


Requires Phoenix 1.0.7.2+

Changes:        
- Added missing final_price to queries. Thanks to @rupruprup for the report

Share this post


Link to post
Share on other sites
On 7/18/2020 at 1:20 PM, raiwa said:

Uploaded update

Version 5.1.3. Phoenix


Requires Phoenix 1.0.7.2+

Changes:        
- Added missing final_price to queries. Thanks to @rupruprup for the report

Hi Rainer,  first of all thank you for the update, timely as usual! I do have one problem though. In PHOENIX 1.0.7.5+ the products bunch up, not responding to the new row display of products and I do not see where I could possibly correct that. 

 

products_by_row.jpg

Share this post


Link to post
Share on other sites

Sorry Rainer,  it does work fine. I reinstalled PHOENIX and installed this addon as the first one and all OK so there must have been a conflict with another addon. Will work it out from there.

Thanks again for all your work!

Share this post


Link to post
Share on other sites

Dear @raiwa!

Thanks a lot for your Recently_Viewed_v5.1.3!

I found a little error:

Notice: Undefined variable: products_query_order in ../includes/modules/content/header/cm_header_recently_viewed.php on line 81

this line:
$products_query = tep_db_query("SELECT DISTINCT " . $products_query_what . $products_query_from . $products_query_where . $products_query_order . "

i fixed it to:

$products_query = tep_db_query("SELECT DISTINCT " . $products_query_what . $products_query_from . $products_query_where . $recently_products_query_order . "

i hope it's right.

Best regards, Vladimir.

Share this post


Link to post
Share on other sites

Uploaded:

Version 5.1.4. Phoenix


Requires Phoenix 1.0.7.2+
Tested with Phoenix 1.0.7.9

Changes:
         - Fixed error in box module

Share this post


Link to post
Share on other sites

Uploaded:

Version 5.1.5. Phoenix


Requires Phoenix 1.0.7.2+
Tested with Phoenix 1.0.7.9

Changes:
   - Fixed undefined variable error in header module

Share this post


Link to post
Share on other sites

Rainer. Just installed this latest version and have the following error on the recently_viewed.php page. I have tried using an older version of this page and the error is stil lthere for some reason.

Quote

1064 - You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ') AND p.products_status = '1' AND p.pr' at line 6

SELECT COUNT(p.products_id) AS total FROM products_description pd, products p LEFT JOIN manufacturers m ON p.manufacturers_id = m.manufacturers_id LEFT JOIN specials s ON p.products_id = s.products_id WHERE p.products_id IN () AND p.products_status = '1' AND p.products_id = pd.products_id AND pd.language_id = '1'

 


REMEMBER BACKUP, BACKUP AND BACKUP

Get the latest Responsive osCommerce CE (community edition) here

It's very easy to over complicate what are simple things in life

Share this post


Link to post
Share on other sites

Reverted back to 7.1 just in case this time,  and the error was still there


REMEMBER BACKUP, BACKUP AND BACKUP

Get the latest Responsive osCommerce CE (community edition) here

It's very easy to over complicate what are simple things in life

Share this post


Link to post
Share on other sites

It seems you still have no recently viewed products in the list:

p.products_id IN (this shouldn't be empty)

Did you directly access the page without having viewed any product?

Share this post


Link to post
Share on other sites

Just checked and it seems to be this. So it's not a real live problem, no one should reach the page with an empty list. However I'll add a check in the next update for any case. Thanks for the report.

If you wish to apply it already:

  require "includes/languages/$language/recently_viewed.php";

  if (empty($_SESSION['recently_viewed'])) {
    tep_redirect(tep_href_link('index.php'));
  }
  
  $recently_viewed = $_SESSION['recently_viewed'];

 

Edited by raiwa

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×