Feature #188

Add debug to the scheme level

Added by patrick_elx over 2 years ago. Updated over 2 years ago.

Status:Closed Start:10/04/2009
Priority:Normal Due date:10/16/2009
Assigned to:tshif % Done:

100%

Category:-
Target version:Caller ID Superfecta v 2.2.1 Maintenance Release Estimated time:1.00 hour

Description

The debug function will test only one scheme at this time (the one on the scheme page that is currently open).

I would suggest that we add a 'test all scheme' checkbox next to the debug button to be able test the number against the scheme matching in addition to only the source lookup process.

page.superfecta.php (16.7 KB) patrick_elx, 10/06/2009 10:29 am

History

Updated by patrick_elx over 2 years ago

seems easy to do.

Just need to add a checkbox and in the function Ht_debug(thenumber)(in page.superfecta.php)

change the line
var poststr = "debug=yes&thenumber=" + thenumber+"&scheme=";

by

var poststr = "debug=yes&thenumber=" + thenumber;
and add to poststr "&scheme=" only if the checkbox is not checked.

Updated by tshif over 2 years ago

  • Due date set to 10/16/2009
  • Status changed from New to Assigned
  • Assigned to set to patrick_elx
  • Target version changed from Caller ID Superfecta - Future Versions to Caller ID Superfecta v 2.2.1 Maintenance Release

I agree. I set the due date to be the same as the projected release of 2.2.1 - let me know if that doesnt work for you.

Updated by patrick_elx over 2 years ago

I'm just not sure how to properly create a checkbox in php.. I'll have to look for it. I'll try to work on it tomorrow.

Updated by patrick_elx over 2 years ago

Uploaded rev.149

Updated by patrick_elx over 2 years ago

  • Status changed from Assigned to QA Testing
  • % Done changed from 0 to 80
  • Estimated time set to 1.00

Updated by tshif over 2 years ago

  • Assigned to changed from patrick_elx to tshif

Updated by tshif over 2 years ago

QS: FUnctions passed QS. Cosmetic Issues remain in fly by help.

dev: Can the flyby help be moved to the left? Currently, hoving over link causes browser to expand to the right. Moving flyby text to the left would correct. Can this be done?

Updated by patrick_elx over 2 years ago

I'm not sure...
However, it should be easy just to put the checkbox below the number box. That should center the flyby help.

Updated by tshif over 2 years ago

  • Status changed from QA Testing to Assigned
  • Assigned to changed from tshif to patrick_elx

Ok - great - you're the man!

Updated by patrick_elx over 2 years ago

updated rev.151

Updated by tshif over 2 years ago

  • Status changed from Assigned to QA Testing
  • Assigned to changed from patrick_elx to tshif

Updated by patrick_elx over 2 years ago

please recheck rev152 there was a conflict with your changes rev 150 and mine rev151.

Updated by tshif over 2 years ago

r150 and 151 work fine for me - what problem do you see?

Updated by tshif over 2 years ago

I just double checked - when I pull r151 it works fine for me - what are you seeing as a failure?
I tested in IE8 and firefox.

Updated by patrick_elx over 2 years ago

The flyby was not working and the size of the police was not consistent with the rest of the page.
But again it could have been a conflict as I was working on 150 while you uploaded 151.
I can try to downgrade again to recheck, but 152 seems to work ok here.

Updated by tshif over 2 years ago

Restorred cosmetic changes (r153) from r150 which were mistakenly removed in r151 and r152

Updated by patrick_elx over 2 years ago

  • File superfectar153.jpg added

Look at the attached picture of the way rev 153 displays on Mozilla on Win XP.
The flyby is not working anymore and is always visible and the checkbox has the wrong size.

Updated by tshif over 2 years ago

Wow - it doesnt do that in my firefox. What version are you using? Ill fix this also, but im curious about your version.

Updated by tshif over 2 years ago

Issue confirmed. correction in progress.

Updated by patrick_elx over 2 years ago

what are you trying to do with the <font=2> </font>?
to make the checkbox smaller or the flyby?

In my case, without the font tag, the size of the checkbox is consistent with the rest of the page, and the flyby has the proper size too.

If you want to reduce the size of the checkbox, just move the </font> out of the <a> </a> statement.

Updated by tshif over 2 years ago

  • % Done changed from 80 to 90

Niether - goal was to make the text `Test all CID schemes` font size 2 as was done for `Check for Data Source File updates online` text.

Change in font characterstics not possible within class="info" without affecting all occurances of that class. SInce thats not desireable, I will learn to live with larger font than I want for this location. Unless you have a way - :)

Updated by patrick_elx over 2 years ago

same problem with me with your rev 154.
Try the attached file, it's working for me with IE and Firefox.

Updated by tshif over 2 years ago

Ugh! I used the wrong (unfixed) page file for r174. Stupid version control problem that I caused. :(

I cant seem to get a clean copy of your attached file but the problem is obvious - as my r174 still had the errant <font> tags in it.

I need to make some citation changes in the page file anyway - so Ill get a proper version in the SVN very soon.

Sorry about the mistake and extra ticket chatter -

Updated by tshif over 2 years ago

Fixed my mistake by using Patricks code. some citation changes.

Patrick - this works ok for me now - please let me know how it is for you.

Updated by tshif over 2 years ago

  • File deleted (superfectar153.jpg)

Updated by tshif over 2 years ago

  • Status changed from QA Testing to Closed
  • % Done changed from 90 to 100

QS: Passed cosmetic and functional tests. No negatives noted.
Patrick - if you find problems, we can reopen this case.

Also available in: Atom PDF