Fixed: "Cannot convert `text` to object" exception in _getRuleText() caused by raw regular expressi…

lib/matcher.js

 /*
  * This file is part of the Adblock Plus extension,
  * Copyright (C) 2006-2012 Eyeo GmbH
  *
  * Adblock Plus is free software: you can redistribute it and/or modify
  * it under the terms of the GNU General Public License version 3 as
  * published by the Free Software Foundation.
  *
  * Adblock Plus is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
@@ skipping 144 matching lines @@
         this._getRuleDomains(filter, rule.options);
         this._getRuleThirdParty(filter, rule.options);
         return rule;
       },
 
       add: function(filter)
       {
         if (filter.text in this._rules)
           return;
 
+        if (!filter.regexpSource) // Regular expressions aren't supported

[Felix Dahlke, 2012/11/14 12:45:26]

So it's a regular expression if regexpSource is falsy?

[Wladimir Palant, 2012/11/14 13:25:30]

Yes - for raw regexps we calculate the regexp property right away, for
everything else we set regexpSource and do it on demand. Relying on regexpSource
here is still a hack, something that we probably need to improve in future.

+          return;
+
         var rule = this._generateRule(filter);
         opera.extension.urlfilter[rule.type].add(rule.text, rule.options);
         this._rules[filter.text] = rule;
       },
 
       remove: function(filter)
       {
         if (!(filter.text in this._rules))
           return;
 
         var rule = this._rules[filter.text];
         opera.extension.urlfilter[rule.type].remove(rule.text);
         delete this._rules[filter.text];
       },
 
       clear: function()
       {
         for (var text in this._rules)
         {
           var rule = this._rules[text];
           opera.extension.urlfilter[rule.type].remove(rule.text);
         }
         this._rules = {};
       }
     }
   };
 })();