Code Database Updates

[Chris]

Jul 23, 2010 1:08:40 GMT -8 SubDevo said:

Thanks for getting to that wormo. But...
Missed a couple parentheses.

var nOrder = (admin)? iStaffMenu : ((pb_username!="Guest") ? iMemberMenu : iGuestMenu);

javascript:alert((0)? iStaffMenu : ( "Guest" !="Guest" ? 1 : 2));

Pretty sure they aren't necessary actually.

[SubDevo]

Yea Chris, I guess they aren't necessary. That is just my habit. lol

[Wormopolis]

<-- joins MAEUP (mothers against extra unnecessary parentheses)!

we have punch and pie.


Thanks for finding that Sub!

[macflurry]

Ross' PM Summary, found here:

support.proboards.com/index.cgi?board=codedatabase&action=display&n=1&thread=70553


After putting it on my forum it works great functionally and appears as it should while reading a PM but when you go to reply the PM summary is squashed to the right of the Send Personal Message box instead of below it.

Eton Bones suggested this fix and it worked a treat:

This looks to be another code affected by the changes made to the standard posting form.

Find this section toward the bottom of the code and remove the red

if(document.postForm && document.postForm.to) {
if(!location.href.match(/ion=pmsend/))
previewMessage();
extractQuote();
document.postForm.onsubmit = function() { saveSummary(); };
document.postForm.parentNode.parentNode.appendChild(document.createElement('br'));
document.postForm.parentNode.parentNode.appendChild(showSummary());
} else if(location.href.match(/ion=pmview/)) {
showMessage();
var table = document.getElementsByTagName('table');
for(t=table.length-1; t>0; t--) {
if(table.item(t).className == 'bordercolor' && table.item(t+1).cellPadding == '4') {
table.item(t).parentNode.insertBefore(showSummary(), table.item(t).nextSibling);
table.item(t).parentNode.insertBefore(document.createElement('br'), table.item(t).nextSibling);
break;
}
}

[Trublu]

Done and noted in the code thread, thanks macflurry.

[Chris]

May 22, 2010 17:16:14 GMT -8 Trublu said:

---

Patch for Cali's Posting Request/Template Form

This patch is apparently no longer needed
(http://studiozero.proboards.com/index.cgi?action=display&board=codesupport&thread=13775&page=1#110870)

Due to Proboards censoring codesupport to coderequests the URL cannot be properly hyperlinked.

[Wormopolis]

Im currently updating UBBC in personal text code

then maybe the "cannot vote in polls" threads will be eiliminated in support.

support.proboards.com/index.cgi?board=codedatabase&action=display&n=1&thread=70696

[Thesealion]

I wasn't sure what this code did and so i put it on my test forum...well it half does something but i don't think it works properly...

I get an error on this line it says "lpp.split(">on ")[1] is undefined"

gtd[2].innerHTML = lpp.split(">on ")[1].split(brr)[0]+"<br />"+lpp.split(brr+"in ")[1];

[Trublu]

Aug 17, 2010 12:02:14 GMT -8 Thesealion said:

I wasn't sure what this code did and so i put it on my test forum...well it half does something but i don't think it works properly...

I get an error on this line it says "lpp.split(">on ")[1] is undefined"

gtd[2].innerHTML = lpp.split(">on ")[1].split(brr)[0]+"<br />"+lpp.split(brr+"in ")[1];

Are you referring to the same code Wormo was talking about, or something else?

[Thesealion]

Sorry i had a total brain fart...edited my post

[Trill]

It seems like the patch that reconfigures the admin panel for CrossFury has broken with the implementation of the new Promotion section. I tried playing around with this code for a little bit, but couldn't get it to work both in IE and other browsers. If anyone could take a look at it, it would be appreciated.

[Thesealion]

Wow....soon it will be time to just get rid of most of the codes and start over...

[Michael]

First CF Fix code should change to:

<script>
/* Patch - Remove ProBoards Blog from Admin Panel */
/* Top of Main Footer */
if(location.href.match(/action=(admin|\w+3)/i)){
var tbl=document.getElementsByTagName('table');
for(var r=tbl.length-1;r>0;r--){
if(tbl[r].rows.length==1 && tbl[r].rows[0].cells.length==4 && /Promote Your Forum/i.test(tbl[r].rows[0].cells[3].innerHTML)){
tbl[r].rows[0].deleteCell(3);
tbl[r].rows[0].cells[2].width="33%";
tbl[r].rows[0].cells[1].width="34%";
tbl[r].rows[0].cells[0].width="33%";
}
}
}
</script>

The 2nd should probably be removed for now...

[Chris]

Aug 21, 2010 7:28:41 GMT -8 Michael said:

First CF Fix code should change to:

<script>
/* Patch - Remove ProBoards Blog from Admin Panel */
/* Top of Main Footer */
if(location.href.match(/action=(admin|\w+3)/i)){
var tbl=document.getElementsByTagName('table');
for(var r=tbl.length-1;r>0;r--){
if(tbl[r].rows.length==1 && tbl[r].rows[0].cells.length==4 && /Promote Your Forum/i.test(tbl[r].rows[0].cells[3].innerHTML)){
tbl[r].rows[0].deleteCell(3);
tbl[r].rows[0].cells[2].width="33%";
tbl[r].rows[0].cells[1].width="34%";
tbl[r].rows[0].cells[0].width="33%";
}
}
}
</script>

The 2nd should probably be removed for now...

Was there an announcement saying the blog would be removed from ACP because I'm confused as to why the addition of forum promotion would invalidate the test for the blog which is still in that cell

[Michael]

Aug 21, 2010 21:54:08 GMT -8 Chris said:

Aug 21, 2010 7:28:41 GMT -8 Michael said:

First CF Fix code should change to:

<script>
/* Patch - Remove ProBoards Blog from Admin Panel */
/* Top of Main Footer */
if(location.href.match(/action=(admin|\w+3)/i)){
var tbl=document.getElementsByTagName('table');
for(var r=tbl.length-1;r>0;r--){
if(tbl[r].rows.length==1 && tbl[r].rows[0].cells.length==4 && /Promote Your Forum/i.test(tbl[r].rows[0].cells[3].innerHTML)){
tbl[r].rows[0].deleteCell(3);
tbl[r].rows[0].cells[2].width="33%";
tbl[r].rows[0].cells[1].width="34%";
tbl[r].rows[0].cells[0].width="33%";
}
}
}
</script>

The 2nd should probably be removed for now...

Was there an announcement saying the blog would be removed from ACP because I'm confused as to why the addition of forum promotion would invalidate the test for the blog which is still in that cell

I'm guessing that it's due to the blog not being the first cell or something ... not sure - either way, the change fixed it!