Jump to content
Sign in to follow this  
scotthatton

Extra pages-info box w/ admin

Recommended Posts

Hi folks,

 

Using the great contribution Extra pages-info box w/ admin

http://www.oscommerce.com/community/contributions,2021

 

However, noticed a problem when linking to the pages when a customer has items in the shopping cart, in that the cart empties.

 

I beleive that the session osCsid is not passed over correctly, as the link includes it as a second ? in the URL, when it should be an &.

 

Anyone know of a quick fix for this? I've done some searching but can't see it mentioned. I'm using version 3.3.

 

Thanks in advance,

Scott

Share this post


Link to post
Share on other sites
Hi folks,

 

Using the great contribution Extra pages-info box w/ admin

http://www.oscommerce.com/community/contributions,2021

 

However, noticed a problem when linking to the pages when a customer has items in the shopping cart, in that the cart empties.

 

I beleive that the session osCsid is not passed over correctly, as the link includes it as a second ? in the URL, when it should be an &.

 

Anyone know of a quick fix for this?  I've done some searching but can't see it mentioned.  I'm using version 3.3.

 

Thanks in advance,

Scott

 

 

Think I've cracked the problem. Seems to be in line 76 where the link is created.

$page_list .= '<tr><td class="infoBoxContents"><a target="'.$target.'" href="' . tep_href_link($link) . '">' . $page['pages_title'] . '</a></td></tr>';

Doesn't used the standard method for making the link as it pulls in a variable from line 63

$link = FILENAME_PAGES . '?pages_id=' . $page['pages_id'];

 

I updated link 76 to:

$page_list .= '<tr><td class="infoBoxContents"><a target="'.$target.'" href="' . tep_href_link(FILENAME_PAGES, 'pages_id=' .$page['pages_id'], 'NONSSL') . '">' . $page['pages_title'] . '</a></td></tr>';

which seems to fix the problem and also makes line 63 redundant.

 

Can anyone confirm this sounds about right? Wondered if there was a reason that I've overlooked for the other way of creating the link.

 

An extra piece of functionality that would be good for the contribution, would be a 3rd type of link. An internal link, but to a set page (specified like an external link).

You could then point it a set pages (such as contact us, catalogue request, all products etc) and maintain the session id. Will have a look if nothing is in the pipeline.

 

Thanks,

Scott

Share this post


Link to post
Share on other sites
Think I've cracked the problem.  Seems to be in line 76 where the link is created. 

$page_list .= '<tr><td class="infoBoxContents"><a target="'.$target.'" href="' . tep_href_link($link) . '">' . $page['pages_title'] . '</a></td></tr>';

Doesn't used the standard method for making the link as it pulls in a variable from line 63

$link = FILENAME_PAGES . '?pages_id=' . $page['pages_id'];

 

I updated link 76 to:

$page_list .= '<tr><td class="infoBoxContents"><a target="'.$target.'" href="' . tep_href_link(FILENAME_PAGES, 'pages_id=' .$page['pages_id'], 'NONSSL') . '">' . $page['pages_title'] . '</a></td></tr>';

which seems to fix the problem and also makes line 63 redundant.

 

Thanks,

Scott

 

Can confirm that this is all running fine on my site. Been live for over a month now. Surprised more people haven't had the problem...

 

BTW, the file to update is the main info_pages.php page.

 

Cheers,

Scott

Share this post


Link to post
Share on other sites
which seems to fix the problem and also makes line 63 redundant.

 

waht do u mean ?

 

by the way it works for me too , i'm posting the fix in the contribution area , but i think that someone should make a complete package , with or without a wysiwyg editor !


MS2

Share this post


Link to post
Share on other sites

i have this contrib. version 4.5

 

my problem is:

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 '-20, 20' at line 12

 

any can help me??

Edited by mipe528

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
Sign in to follow this  

×