Jump to content
  • Checkout
  • Login
  • Get in touch

osCommerce

The e-commerce.

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

Link to comment
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

Link to comment
Share on other sites

  • 1 month later...
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

Link to comment
Share on other sites

  • 9 months later...
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

Link to comment
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
Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Unfortunately, your content contains terms that we do not allow. Please edit your content to remove the highlighted words below.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...