Author |
Message |
pitfighter
New Member


Joined: Aug 17, 2005
Posts: 20
|
Posted:
Fri Aug 19, 2005 6:05 am |
|
lol wished my girlfriend would think the same  |
|
|
|
 |
hitwalker
Sells PC To Pay For Divorce

Joined:
Posts: 5661
|
Posted:
Fri Aug 19, 2005 6:12 am |
|
well ladies aren't satisfied that easy.....lol |
|
|
|
 |
Raven
Site Admin/Owner

Joined: Aug 27, 2002
Posts: 17088
|
Posted:
Fri Aug 19, 2005 7:11 am |
|
First of all, it's not java, so there is nothing to gain from messing with your java runtime. It's JavaScript and is a function of the browser.
Now, with that out of the way, please provide a link so I can take a look. |
|
|
|
 |
Raven

|
Posted:
Fri Aug 19, 2005 7:27 am |
|
I just went to the link that Hitwalker supplied me, in IE6, and I dont see any error, at least on the home page. Could it be the particular theme you are/were using? |
|
|
|
 |
pitfighter

|
Posted:
Fri Aug 19, 2005 9:15 am |
|
hi,
i tried different themes and with all themes i get that error. error only shows when clicking on any sentinal admin link...strange thing of it is that when i get my mouse over one of the little red help boxes the error dissapears, but when clicked on one of the sentinaladmin links the error returns |
|
|
|
 |
Raven

|
Posted:
Fri Aug 19, 2005 10:10 am |
|
Please PM me an adminid/pass so I can debug this for you. |
|
|
|
 |
Raven

|
Posted:
Fri Aug 19, 2005 12:49 pm |
|
Open language/nukesentinel/lang-english.php and at or around line 242 you will see this codeCode:define("_AB_HELP_078","You do not have the IP 2 Country table data installed.<br />This means your are missing the IANA Reserved Ranges that should be blocked.<br />Please use the <a href='".$admin_file.".php?op=ABImport'>Import Data</a> section to install the data.");
|
Just to test that this is the line that is causing the error, change it toCode:define("_AB_HELP_078","You do not have the IP 2 Country table data installed.");
|
Did that fix it? |
|
|
|
 |
pitfighter

|
Posted:
Fri Aug 19, 2005 1:02 pm |
|
that did the job....
its working now raven! error is gone..
just curious...how did you now ? and whats causing the error ?
Thanks |
|
|
|
 |
Raven

|
Posted:
Fri Aug 19, 2005 1:10 pm |
|
This is the first chance I had to dig into this. As soon as you gave me access and I saw the error, I did a view page source and looked at line 125. When I saw mixed quotes I had a suspicion. Over many years of working with JavaScript, I know how persnickety it is when blending "" and ' in the status window. So, from there it was a simple matter of proving my suspicion. |
|
|
|
 |
pitfighter

|
Posted:
Fri Aug 19, 2005 1:17 pm |
|
to be honest i had give up on it... i never would find that error...
thanks for all the help! |
|
|
|
 |
Raven

|
Posted:
Fri Aug 19, 2005 1:19 pm |
|
|
|
 |
tangoman
Involved


Joined: Aug 06, 2005
Posts: 301
|
Posted:
Fri Aug 19, 2005 1:19 pm |
|
Raven,
As you will recall, subsequent to me highlighting exactly the same issue earlier today, you forwarded details of this link/solution.
I too have now replaced the above code and everything is now working correctly, without any error being displayed!!!
I do note that although you advised the code requring replacement was:
define("_AB_HELP_078","You do not have the IP 2 Country table data installed.<br />This means your are missing the IANA Reserved Ranges that should be blocked.<br />Please use the <a href='".$admin_file.".php?op=ABImport'>Import Data</a> section to install the data.");
The code I actually had in my file was slightly different:
define("_AB_HELP_078","You do not have the IP 2 Country table data installed. Please use the <a href='".$admin_file.".php?op=ABImport'>Import Data</a> section to install the data.");
I wonder if this had anything to do with it? |
_________________ I am using PHPNuke Version 7.6 with patch 3.0, CNB YA 4.4.2, NukeSentinel(tm) 2.3.2 and Nuke Royal. |
|
|
 |
Raven

|
Posted:
Fri Aug 19, 2005 1:23 pm |
|
Nope. Are you using the latest NukeSentinel? The issue, regardless of the symantecs, is the mixing of quotes. |
|
|
|
 |
hitwalker

|
Posted:
Fri Aug 19, 2005 1:25 pm |
|
nice raven...
well i was close but i didnt saw original source as it is...
but i wouldnt noticed anyway...
and pitfighter,remember the edits....next update/upgrade your likely to overwrite files... |
|
|
|
 |
tangoman

|
Posted:
Fri Aug 19, 2005 1:29 pm |
|
Raven,
My updated signature should answer that for you...(Thanks for 'pointing that out')!
 |
|
|
|
 |
pitfighter

|
Posted:
Fri Aug 19, 2005 1:30 pm |
|
hitwalker wrote: | nice raven...
well i was close but i didnt saw original source as it is...
but i wouldnt noticed anyway...
and pitfighter,remember the edits....next update/upgrade your likely to overwrite files... |
sorry dont follow...you mean i had to overwrite those files ? |
|
|
|
 |
hitwalker

|
Posted:
Fri Aug 19, 2005 1:32 pm |
|
no....lol
i mean...you edited file(s) of sentinel,next update or upgrade of sentinel watch out you dont overwrite the edited file(s). |
|
|
|
 |
tangoman

|
Posted:
Fri Aug 19, 2005 1:35 pm |
|
hitwalker,
What are you smoking?! .......
You are talking hyrogliphics to ME now!...  |
|
|
|
 |
pitfighter

|
Posted:
Fri Aug 19, 2005 1:38 pm |
|
aaah okay lol
think that will not happen, but on the otherside i liked that re-installing of nuke + patched + sentinal (6 times)
 |
|
|
|
 |
tangoman

|
Posted:
Fri Aug 19, 2005 1:40 pm |
|
SIX?!!!!...........
Kevin!!!...pitfighter beat me by TWO!!!.......  |
|
|
|
 |
hitwalker

|
Posted:
Fri Aug 19, 2005 1:41 pm |
|
Smoking?
He...he...i dont drink anymore ,i dont smoke anymore ....  |
|
|
|
 |
tangoman

|
Posted:
Fri Aug 19, 2005 1:42 pm |
|
Oh I do!....When I'm on fire!!!..... ..........  |
|
|
|
 |
BobMarion
Former Admin in Good Standing

Joined: Oct 30, 2002
Posts: 1037
Location: RedNeck Land (known as Kentucky)
|
Posted:
Fri Aug 19, 2005 2:50 pm |
|
okay, I know why it caused the error. The single quote marks should have been escaped like so:Code:define("_AB_HELP_078","You do not have the IP 2 Country table data installed.<br />This means your are missing the IANA Reserved Ranges that should be blocked.<br />Please use the <a href=\'".$admin_file.".php?op=ABImport\'>Import Data</a> section to install the data.");
|
|
_________________ Bob Marion
Codito Ergo Sum
Only registered users can see links on this board! Get registered or login! |
|
|
 |
tangoman

|
Posted:
Fri Aug 19, 2005 2:53 pm |
|
Hey Bob,
Thanks for the reply.
So should we stick with Ravens original solution, or use your code instead? |
|
|
|
 |
BobMarion

|
Posted:
Fri Aug 19, 2005 3:21 pm |
|
Either will work, the download has been updated with the one posted to prevent future problems  |
|
|
|
 |
|