[RD] Code Fix Report

Excellent! And you're welcome. I've been using this code and it's working fine on its newly updated state. ^_^

I guess, I've been the only one recently who's been most eager with using this code. ;D

Good day to ya Triad!



Ok, I guess I have been poked enough to come and reply to this! lol
The deal with the Original Ross code is...

Problem#1...
it doesn't give a rank to anyone under 10 posts. The couple lines of code Lunatic Eyes posted takes care of that.

Problem #2...
If you are using Group images for staff (or any other group for that matter), each time a new posting rank is reached the Group image is changed to whatever rank is in the Ross code.

Instead of rewritting his code (no permission here!), I just wrote a "fix" code. It very simply, changes the "Gender:" to "Gender :" for any affected users in the list. That small change bypasses the Ross code, so group images work normally. Sweet.

Place this code directly above the "Ross" code.
Be aware, that if you change anything other than "Gender", you may have to move any other codes that rely upon a certain word to be found in the mini-profile in order to work, must be placed ABOVE this code.
Example code is "Level up meter". It needs to find the word "Posts:"

Variable Description: No touchie the Red parts. Edit the Maroon parts.

var fStaff="admin|staff3|staff2|staff3"; // Forum staff usernames.
Enter the affected member(s) user (not display) name(s) on this line.
Separate the names with a "|". Do not put a "|" after the last name.

var fReplace=[ // DO NOT TOUCH
   ["Gender:","Gender :"] // "Text to Replace", "Replacement".
];
Just what it says. Put the text you wish to replace in the first option. And the replacement in the second option. Both of these are case sensitive.

Location: Global Footer above "Ross" code

Code:

<script type="text/javascript">
<!--
/* Staff Fix for "Ranks Based on a Users Gender by Ross" by SubDevo */
/* Global Footer - Please leave this header intact. Do not repost. */
/* interoceandesigns.proboards.com or lsdp.proboards.com */

var fStaff="admin|staff3|staff2|staff3"; // Forum staff usernames.
var fReplace=[ // DO NOT TOUCH
   ["Gender:","Gender :"] // "Text to Replace", "Replacement".
];

var tr=document.getElementsByTagName("tr"); var nf="";
var nP=/n=viewprofile&/; var nx= new RegExp("user=("+fStaff+")","i");
if(location.href.match(/=(display|(pm)?view|user|recent)/i)||document.title.match(/Search/)||document.postForm){
   for(x=0;x<tr.length;x++){with(tr[x]){ var nN=firstChild; var nI=nN.innerHTML;
      if(nN&&nN.width=="20%"&&nI.match(nx)&&nI.match(nP)){
         for(var i=0;i<fReplace.length;i++){nI=nI.replace(fReplace[0],fReplace[1]);}
         nN.innerHTML=nI;
      }
   }}
}
//-->
</script>

Now, if you haven't noticed...
This code isn't just a "fix" code. You can use it to replace anything in the mini-profile for certain users only.
Very powerful if you use/know RegEx expressions...

Note to coders:
Notice I didn't grab all the "td"s? I used the firstChild of each "tr".
Reasons are:
"tr" array has fewer members. Shorter, faster loop.
Each "tr" MUST have, at least, ONE "td" (HTML standard). So "firstChild" is always valid.


Regards,
SubDevo

Thanks Devo for doing all this, but it would probably be easier on everyone to just re-make the code. If you want to then feel free to re-make it and submit it to the database, but I'll link your post in code's thread for now. Ross is no longer around, and I don't think he would be angry if you re-did it (if he is then just link to this post). I'm sure if he took another look at it he would want to re-write it himself. As I've said before, most scripts on Proboards are way too simple and short to be copyrighted.

And by the way, the reason I haven't been updating the database is because my college's network blocks the headers and footers so I cannot update it. Hopefully Todge will get some motivation to do the dreadful task.