Issue 1170 - [adblockplus.org Anwiki to CMS migration] Migrate content

filters/parse_interface.py

 # This file is part of the Adblock Plus website,
 # Copyright (C) 2006-2015 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
 # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
@@ skipping 22 matching lines @@
           if argument.strip():
             match = re.match(r"^\s*(%s)\s+(\S+)\s*$" % TYPE_REGEXP, argument)
             if not match:
               warnings.warn("Skipped malformed argument: '%s'" % argument)
               continue
             argument_type, argument_name = match.groups()
             arguments.append({
               "name": argument_name,
               "type": argument_type
             })
-      parsed.append({
+      value.update({
         "type": "method",
         "name": property_name,
-        "version": value.get("version", ""),
         "return_type": return_type,
         "arguments": arguments
       })

[Wladimir Palant, 2015/03/06 23:21:06]

With my changes value has a lot more properties and these properties need to be
kept. My suggestion would be to add properties to value rather than creating a
new dictionary. So value.update({...}) here and then parsed.append(value). Same
for properties below.

+      parsed.append(value)
     else:
       # Item is a property
       match = re.match(r"^\s*(readonly\s+)?(%s)\s+(\S+)\s*$" % TYPE_REGEXP, key)
       if not match:
         warnings.warn("Skipped malformed property: '%s'" % key)
         continue
       property_modifier, property_type, property_name = match.groups()
-      parsed.append({
+      value.update({
         "type": property_type,
         "name": property_name,
         "modifier": property_modifier or ""
       })
+      parsed.append(value)
   parsed.sort(key=lambda x: x["name"])
   return parsed