| Author |
Message |
hitwalker Sells PC To Pay For Divorce

Joined: Posts: 5661
|
Posted:
Tue Jul 26, 2005 2:22 pm |
|
Hi bob,
yeah long time ago since i ever saw an error with your scripts bob but cant help it now....lol
Ok story is ,i changed to pached 3.0 and suddenly your news popup window shows...
Fatal error: Cannot redeclare opentable() (previously declared in /themes/fiappleblue210/tables.php:15) in /themes/fiappleblue210/tables.php on line 14
This shows with my current theme and with nuke standards deepblue.
Naturaly when i set the window setting to "page" it shows everything normal.
Any idea? |
|
|
|
 |
Raven Site Admin/Owner

Joined: Aug 27, 2002 Posts: 16987 Location: Kansas
|
Posted:
Tue Jul 26, 2005 4:27 pm |
|
In your themes/fiappleblue210/theme.php file, try changing | Code: | | include("themes/fiappleblue210/tables.php"); |
to | Code: | | include_once("themes/fiappleblue210/tables.php"); |
|
|
|
|
 |
hitwalker Sells PC To Pay For Divorce

Joined: Posts: 5661
|
Posted:
Tue Jul 26, 2005 4:45 pm |
|
hi raven,
well nice idea but it moved the problem to the themeheader function..
| Code: |
Fatal error: Cannot redeclare themeheader() (previously declared in /themes/fiappleblue210/theme.php:33) in /themes/fiappleblue210/theme.php on line 32 |
|
|
|
|
 |
Raven Site Admin/Owner

Joined: Aug 27, 2002 Posts: 16987 Location: Kansas
|
Posted:
Tue Jul 26, 2005 4:57 pm |
|
Well, you have the general idea - Happy hunting! See also this post |
|
|
|
 |
hitwalker Sells PC To Pay For Divorce

Joined: Posts: 5661
|
Posted:
Tue Jul 26, 2005 5:01 pm |
|
ah thanks,that looks interesting.
gonna check that out...  |
|
|
|
 |
hitwalker Sells PC To Pay For Divorce

Joined: Posts: 5661
|
Posted:
Tue Jul 26, 2005 5:20 pm |
|
well to bad raven,i hoped that was gonna solve it but no...
it sure looked as it was the same error but it didnt help..
Bob are you in the area...?...lol |
|
|
|
 |
dean Worker


Joined: Apr 14, 2004 Posts: 193
|
Posted:
Wed Jul 27, 2005 1:26 am |
|
I see the same error when logging out of a theme that is not the default theme for the site. I am running patch 3.0b, Sentinel 2.32 on nuke 7.3. I tried both of the include_once edits in both the related theme and Sentinel print files to no avail. Save me Nuke Gods!!!!! |
|
|
|
 |
chatserv The Mouse Is Extension Of Arm

Joined: May 02, 2003 Posts: 1396 Location: Puerto Rico
|
Posted:
Wed Jul 27, 2005 2:09 am |
|
I'm thinking this is the code that creates the popup window (in the News module's index.php):
| Code: | include("header.php");
if($neconfig['readmore'] == 1) {
echo "<script language='Java Script'>\n";
echo "<!-- Begin\n";
echo "function NewsReadWindow(mypage, myname, w, h, scroll) {\n";
echo "var winl = (screen.width - w) / 2;\n";
echo "var wint = (screen.height - h) / 2;\n";
echo "winprops = 'height='+h+',width='+w+',top='+wint+',left='+winl+',scrollbars='+scroll+''\n";
echo "win = window.open(mypage, myname, winprops)\n";
echo "if (parseInt(navigator.appVersion) >= 4) { win.window.focus(); }\n";
echo "}\n";
echo "// End -->\n";
echo "</ script>\n";
} |
If that's the case i would move the header.php include line to the end of it:
| Code: | if($neconfig['readmore'] == 1) {
echo "<script language='Java Script'>\n";
echo "<!-- Begin\n";
echo "function NewsReadWindow(mypage, myname, w, h, scroll) {\n";
echo "var winl = (screen.width - w) / 2;\n";
echo "var wint = (screen.height - h) / 2;\n";
echo "winprops = 'height='+h+',width='+w+',top='+wint+',left='+winl+',scrollbars='+scroll+''\n";
echo "win = window.open(mypage, myname, winprops)\n";
echo "if (parseInt(navigator.appVersion) >= 4) { win.window.focus(); }\n";
echo "}\n";
echo "// End -->\n";
echo "</ script>\n";
}
include("header.php"); |
Worth trying (i added a space to java script and / script to avoid triggering NukeSentinel). |
|
|
|
 |
Raven Site Admin/Owner

Joined: Aug 27, 2002 Posts: 16987 Location: Kansas
|
Posted:
Wed Jul 27, 2005 2:15 am |
|
And/Or change it to include_once("header.php"); |
|
|
|
 |
BobMarion Former Admin in Good Standing

Joined: Oct 30, 2002 Posts: 1043 Location: RedNeck Land (known as Kentucky)
|
Posted:
Wed Jul 27, 2005 2:46 am |
|
In modules/News/read_article.php find: | Code: | | include("themes/$Theme_Sel/theme.php"); |
Change it to: | Code: | | include_once("themes/$Theme_Sel/theme.php"); |
|
|
|
|
 |
hitwalker Sells PC To Pay For Divorce

Joined: Posts: 5661
|
Posted:
Wed Jul 27, 2005 4:39 am |
|
well how am i gonna bring it....mmmm
thanks cs...for breaking your head over it...lol..
thanks raven but the winner is bob....
and like to say its stupid i didnt thought of the read_article.php before.
I played around with the file before as bob knows to include google adsense.
should have looked into that file instead of the tables and/or header... |
|
|
|
 |
chatserv The Mouse Is Extension Of Arm

Joined: May 02, 2003 Posts: 1396 Location: Puerto Rico
|
Posted:
Wed Jul 27, 2005 9:33 am |
|
You learn something new everyday  |
|
|
|
 |
|
|
|
|