Jump to content


Corporate Sponsors


Latest News: (loading..)

- - - - -

Contribution Issue


366 replies to this topic

#361 GoTTi

  • Community Member
  • 438 posts
  • Real Name:GoTTi

Posted 07 April 2012, 17:59

this is not in index.php


REPLACE
    if (tep_not_null(TEXT_MAIN)) {
?>
  <div class="contentText">
    <?php echo TEXT_MAIN; ?>
  </div>
<?php
    }
WITH
echo $pagetext;


#362 al3ks

  • Community Member
  • 104 posts
  • Real Name:Aleksander
  • Gender:Male

Posted 09 April 2012, 02:22

Hello everyone,

I have installed this contribution, and everything works ok but I don't know how to access the created pages with this contribution.

Any advice?

#363 jhaugen

  • Community Member
  • 20 posts
  • Real Name:Jerry Haugen

Posted 10 April 2012, 05:59

I installed the 31 March 2012 version of this on a 2.3.1 store after installing CKEditor. Almost everything works fine after I modified the 'pages' table to make page_type a char(2) instead of Char(1) (thus getting up to 99 pages).

My two issues are:

1. contact_us.php is not getting into the page I create for it. Editing the home page works fine (the new additiions to the store show up under the stuff I added with this module), but I left it with page_type = 1. I want to have contact_us have page_type=11. The data on it is in both tables correctly, as far as I can tell. I edited contact_us.php as per the instructions except with page_type = 11. I can put stuff into the page via the admin tool, but the standard Contact Us form does not appear (actually nothing from contact_us.php appears). Apparently extra_info_pages.php is not connecting to contact_us.php at all. Seems like it should be something simple I'm missing. Any ideas??

2. I cannot make the text box for the external link active (on any of my pages) - even after clicking the 'External' button. The source code for the page says 'disabled' in that part of the form. Any ideas??


Thanks,
-Jerry-

#364 jhaugen

  • Community Member
  • 20 posts
  • Real Name:Jerry Haugen

Posted 11 April 2012, 06:52

I resolved my issue concerning the inability to set an external link. For whatever reason the value for "intorext" in the database table "pages_description" for my page was set to zero. This value tells the system to use an external link (if set to 1) or not (if set to 0). Clicking the radio buttons had no effect although they should toggle. After editing the table to set the value of "intorext" to 1 rather than 0, I was able to put my link in the box, and update the page. This put the link into the database and all is well. I don't like to mess with the tables directly if I can help it, so I experimented further.

I found that if I
  • click the "External" radio button
  • click update
  • edit the page again
the link can be entered into the box. Click update again and the external link works fine.

Apparently there is no automatic update of the database when the radio button is clicked - at least in the 31 March 2012 version.

-Jerry-

#365 jhaugen

  • Community Member
  • 20 posts
  • Real Name:Jerry Haugen

Posted 11 April 2012, 07:32

I came up with a workaround for my issue with the Contact Us page (the workaround is described at the bottom of this post) and I think I have identified a problem with the code in catalog/includes/modules/boxes/bm_extrainfopage.php I find the following code segment there:

while ($page = tep_db_fetch_array($page_query)) {
   $rows++;
   $target="";
   if($page['link_target']== 1)  {
	$target="_blank";
   }
   if($page['pages_title'] != 'Contact Us'){
	$link = FILENAME_PAGES . '?pages_id=' . $page['pages_id'];
   }else{
	$link = FILENAME_CONTACT_US;
   }
   if($page['intorext'] == 1)  {
	$data .= '&amp;--#60;tr&amp;--#62;&amp;--#60;td class="bg_list2"&amp;--#62;&amp;--#60;a target="'.$target.'" href="' . $page['externallink'] . '"&amp;--#62;' . $page['pages_title'] . '&amp;--#60;/a&amp;--#62;&amp;--#60;/td&amp;--#62;&amp;--#60;/tr&amp;--#62;';
   }else {
	$data .= '&amp;--#60;tr&amp;--#62;&amp;--#60;td class="bg_list2"&amp;--#62;&amp;--#60;a target="'.$target.'" href="' . tep_href_link(FILENAME_PAGES, 'pages_id=' .$page['pages_id'], 'NONSSL') . '"&amp;--#62;' . $page['pages_title'] . '&amp;--#60;/a&amp;--#62;&amp;--#60;/td&amp;--#62;&amp;--#60;/tr&amp;--#62;';
   }

I'm not expert enough to be editing code like this, but it looks to me like the $link string is being set up to define the page to display, but it is never used. If the title of the page is "Contact Us" $link gets assigned contact_us.php. The next 'if' clause goes to an external link (if intorext = 1) or to a page. Seems like if $link were used for the page rather than
FILENAME_PAGES, 'pages_id=' .$page['pages_id']
Contact Us would work properly. It's not a very good solution because if someone wanted to call the link "Write to Us" or something else, it wouldn't work.

I'd appreciate it if someone with more experience with this would take a look and see if that's the problem and fix it with an update. Perhaps it would be best to leave Contact Us out of it completely and provide instructions to have people load the page as I did in my workaround described below.

Here is a workaround:

Use Extra Info Pages to edit the Contact Us page and set /catalog/contact_us.php or /contact_us.php (depending upon where the file is located) as an external link. Note that this version of contact_us.php should NOT be edited with the instructions provided in Extra Info Pages installation file.

-Jerry-

Edited by jhaugen, 11 April 2012, 07:37.


#366 jhaugen

  • Community Member
  • 20 posts
  • Real Name:Jerry Haugen

Posted 11 April 2012, 23:34

I also found another error in catalog/includes/modules/boxes/bm_extrainfopage.php

This code:
if($page['intorext'] == 1)  {
                $data .= '<tr><td class="bg_list2"><a target="'.$target.'" href="' . $page['externallink'] . '">' . $page['pages_title'] . '</a></td></tr>';
            }else {
                $data .= '<tr><td class="bg_list2"><a target="'.$target.'" href="' . tep_href_link(FILENAME_PAGES, 'pages_id=' .$page['pages_id'], 'NONSSL') . '">' . $page['pages_title'] . '</a></td></tr>';
            }
outputs html with a null target (i.e. <a target="" . . . </a>) when displaying the list of links in the info box. This generates an html error for every link (except for external links where the value of $target is "_blank"). The fix is to change this:
}else {
                $data .= '<tr><td class="bg_list2"><a target="'.$target.'" href="' . tep_href_link(FILENAME_PAGES, 'pages_id=' .$page['pages_id'], 'NONSSL') . '">' . $page['pages_title'] . '</a></td></tr>';
            }
to this
}else {
                $data .= '<tr><td class="bg_list2"><a  href="' . tep_href_link(FILENAME_PAGES, 'pages_id=' .$page['pages_id'], 'NONSSL') . '">' . $page['pages_title'] . '</a></td></tr>';
            }

That is, just remove the target attribute from the else clause that lists all the links that are not external.

#367 ce7

  • Community Member
  • 10 posts
  • Real Name:lyn

Posted 20 May 2012, 14:17

I download several version since 2011-08-14,
only this one show link in backend admin, but it did not show me the page type or options,
and it make the contact us in the front end web page funny!

Tried to reinstalled with 2012-03-31 version,
but then it said catalog/extra_info_pages.php fatal error with Line 509

Is there any update full package that actually works ok?