[Version 1.8] Website Enhancement Script

[Kip]

Thanks for the great script.  I find it very convenient to be able to store links to various often-used pages in the side bar.

---

I have a small suggestion regarding the way the links are stored.  This change would allow links which contain commas (such as forum threads) to be specified.

Instead of using [tt]join()[/tt] and [tt]split(",")[/tt] to create and display the list of links, use [tt]join("|")[/tt] and [tt]split("|")[/tt].  As it is now, if there is a comma in a link, the script splits the link at the comma causing the list to display incorrectly.  By using the [tt]|[/tt] symbol to separate the links, this can be avoided.

To implement this change, edit the [tt]join[/tt] and [tt]split[/tt] methods on lines 96, 114, 132, 169, 186, 198, 222, 232, and 242.

---

Also, I noticed something about the way the website handles the crafting pages.  It seems that the website uses a "p=" parameter to determine which character receives the crafting experience points.  All the crafting pages for my first character end in [tt]p=1[/tt].  All the crafting pages for my second character end in [tt]p=2[/tt].  If the wrong number is used, the item is crafted but the experience points are not gained.  This means, for example, that if I wanted to store a link for salvaging metal in the side bar, it would only work for one of the characters.

So far, I have not come up with any possible solutions besides saving a separate link for each character.  I don't really consider this a problem with the script - it is just unfortunate that the website requires this parameter.  Just wanted to mention this for those who have multiple characters.

---

Thanks again for the script.

[Tennenoth]

Thank you for your feedback Kip.

With regards to the comma on the split, I will slap Jikuu for that one and change it as soon as my exams are over.

Secondly, before I hit the exams deadline, I was implementing an "auto-character login/switcher" so once that is in, it would be nice and simple to just switch the number with regards to which character you're on.

Otherwise, I would say that the script would be able to dynamically store variables for each character, so that when you switch to an alt, you'll have completely different links than to your main. (With regards to crafting, others will be the same). That shouldn't be too difficult to implement and would really solve the issue and allow for as many characters with links as you want.

To be honest, with only having one character with crafting, I never thought about what would happen in that situation, but I am glad you brought that to light Kip.

Thanks for that one, and as I say, I will start work (and kick Jikuu into shape) once we finish our exams. Mine, as a reference will finish on the 25th this month.

[Birdtalon]

Good luck! 

[Senni]

Good luck , mine also finish on 25th xD

[Hypernoma]

Mine also finish on the 25th... I smell a conspiracy.

[Chris]

Good luck guys, mine finish on the 21st of June.

[Tennenoth]

Good luck guys, mine finish on the 21st of June.

Ooof, I don't envy you my friend, as the same to the rest of you, good luck/I hope you did well and thanks for your support!

[Kip]

I noticed that the "NI EXP Le Lazy - Jikuu_Ryuu" section is slightly broken because of the removal of mounted xp.  This causes the script to not calculate how much assist xp to use for gold.

To correct the problem, replace this code...

[code=Original Code]
function addEvents() {
obj.getElementsByClassName('a2')[0].addEventListener('click', function(){inputEXP('assist_ranged')}, false);
obj.getElementsByClassName('a3')[0].addEventListener('click', function(){inputEXP('assist_melee')}, false);
obj.getElementsByClassName('a4')[0].addEventListener('click', function(){inputEXP('assist_mounted')}, false);
obj.getElementsByClassName('a5')[0].addEventListener('click', function(){inputEXP('assist_gold')}, false);
}
[/code]

with this code...

[code=New Code]
function addEvents() {
obj.getElementsByClassName('a2')[0].addEventListener('click', function(){inputEXP('assist_ranged')}, false);
obj.getElementsByClassName('a3')[0].addEventListener('click', function(){inputEXP('assist_melee')}, false);
obj.getElementsByClassName('a4')[0].addEventListener('click', function(){inputEXP('assist_gold')}, false);
}
[/code]

(The function starts on line 303.)

[Tennenoth]

I noticed that the "NI EXP Le Lazy - Jikuu_Ryuu" section is slightly broken because of the removal of mounted xp.  This causes the script to not calculate how much assist xp to use for gold.

To correct the problem, replace this code...

[code=Original Code]
function addEvents() {
obj.getElementsByClassName('a2')[0].addEventListener('click', function(){inputEXP('assist_ranged')}, false);
obj.getElementsByClassName('a3')[0].addEventListener('click', function(){inputEXP('assist_melee')}, false);
obj.getElementsByClassName('a4')[0].addEventListener('click', function(){inputEXP('assist_mounted')}, false);
obj.getElementsByClassName('a5')[0].addEventListener('click', function(){inputEXP('assist_gold')}, false);
}
[/code]

with this code...

[code=New Code]
function addEvents() {
obj.getElementsByClassName('a2')[0].addEventListener('click', function(){inputEXP('assist_ranged')}, false);
obj.getElementsByClassName('a3')[0].addEventListener('click', function(){inputEXP('assist_melee')}, false);
obj.getElementsByClassName('a4')[0].addEventListener('click', function(){inputEXP('assist_gold')}, false);
}
[/code]

(The function starts on line 303.)

Yes indeed.
Thanks for saving me time and spurring me to actually sort this thing out. I've been rather busy if you haven't guessed

[Kip]

I've been rather busy if you haven't guessed

I was guessing you might have been.  I've had a fairly busy summer myself.

If there is anything I can do to help with the script, let me know and I'll try my best to help.  (I don't have a lot of JavaScript experience, but I'm working on it and can usually get some code working when I have a specific goal.)  Meanwhile, if I find anything else, I will post about it.