RJTPlomp Posted August 21, 2006 Share Posted August 21, 2006 Checking or unchecking the check boxes (right) next to the category name don't work anymore in 5.3.RolandDordrecht, The Netherlands Link to comment Share on other sites More sharing options...
zorphnog Posted August 21, 2006 Share Posted August 21, 2006 @RJTPlomp: The category check box will check apps, but not uncheck them.Checking a category box will check any unchecked items within that category. However, unchecking the category does not uncheck all the apps in that category. This may be something that needs to be changed.Good catch RJTPlomp. Link to comment Share on other sites More sharing options...
zorphnog Posted August 21, 2006 Share Posted August 21, 2006 Definitely a bug. Here's the fix.In check.js:Function setUncheckedelse { var chkbox = document.getElementById("chkbox"+i); var lbl = document.getElementById('lbl'+i); if (chkbox==null) return; if (forc[i]!=null && forc[i]=='yes') //<------------add this line return; //<------------add this line chkbox.checked=false; lbl.className='txt'; tabs++; uncheckChildren(i); if (DisableOnDepsNotMet) disableChildren(i); if (useExclusions) checkExclusions(i); tabs--;}Function checkCategoryif (isCategory(thisChk)) //If category checkbox was checked{ thisCat = thisChk; for(i=1; cat[i] != null; i++) { if (cat[i] == thisCat) { if (state) { setChecked(i); checkDeps(i); } else {// setUnchecked(i); //<--------remove the "//" at the begining checkDeps(i); } } }} Link to comment Share on other sites More sharing options...
mritter Posted August 21, 2006 Share Posted August 21, 2006 See this reply:http://www.msfn.org/board/index.php?s=&...st&p=550576I disabled the fully un-check a category on purpose. It would cause forced items to become un-checked. Seems like a long-way-around-fix then manually uncheck items, but it prevents a lot of headaches. Link to comment Share on other sites More sharing options...
mritter Posted August 22, 2006 Share Posted August 22, 2006 @Zorphnog: We were both correct. With all 3 updates together it works 100% again. I mis-read where you added the 2 lines. Attached is the working check.js. Link to comment Share on other sites More sharing options...
zorphnog Posted August 22, 2006 Share Posted August 22, 2006 Ok, good. I was starting to get confused as to what you were talking about. Link to comment Share on other sites More sharing options...
RJTPlomp Posted August 25, 2006 Author Share Posted August 25, 2006 (edited) Ok, good. I was starting to get confused as to what you were talking about.I was confused as well... but downloaded the new fix, tested it, and found out it works like magic again. Great, thx!Ro Edited August 25, 2006 by RJTPlomp Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now