[Pkg-privacy-commits] [xul-ext-monkeysphere] 223/296: fix cert checking in clearSite function.

[Ximin Luo]

This is an automated email from the git hooks/post-receive script.

infinity0 pushed a commit to branch debian
in repository xul-ext-monkeysphere.

commit ce03a3a4e3f8dc04e2943a1cd84c3071354f64a6
Author: Jameson Rollins <jrollins at finestructure.net>
Date:   Sat May 1 15:06:16 2010 -0400

    fix cert checking in clearSite function.
    
    The problem was bad logic in the cert check.
---
 Changelog                      | 4 ++--
 chrome/content/monkeysphere.js | 5 ++---
 2 files changed, 4 insertions(+), 5 deletions(-)

diff --git a/Changelog b/Changelog
index 930a35c..735db9f 100644
--- a/Changelog
+++ b/Changelog
@@ -1,8 +1,8 @@
 xul-ext-monkeysphere (0.3~pre) unstable; urgency=low
 
-  * Fix clearSite status menu function **NOT FIXED YET**
+  * Fix clearSite status menu function
 
- -- Jameson Rollins <jrollins at finestructure.net>  Thu, 29 Apr 2010 12:39:57 -0400
+ -- Jameson Rollins <jrollins at finestructure.net>  Thu, 01 May 2010 15:04:57 -0400
 
 xul-ext-monkeysphere (0.2); urgency=low
 
diff --git a/chrome/content/monkeysphere.js b/chrome/content/monkeysphere.js
index 4482118..36b54d8 100644
--- a/chrome/content/monkeysphere.js
+++ b/chrome/content/monkeysphere.js
@@ -319,9 +319,8 @@ var monkeysphere = (function() {
         var apd = ms.createAgentPostData(uri, cert);
         apd.log();
         ms.overrides.clear(apd);
-        // FIXME: why does the override seem to persist after a clear?
-        // this is a pretty big problem
-        if(!ms.overrides.certStatus(apd)) {
+        if(ms.overrides.certStatus(apd)) {
+          ms.log("**** WARNING: override cert not cleared ****");
           alert('Monkeysphere error: override cert not cleared!');
         }
         var newstate = "CLEARED";

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-privacy/packages/xul-ext-monkeysphere.git