[plugin|licenses] Do not override engine methods

Fixes: #5

addons/licenses/component.gd

@@ -16,14 +16,19 @@ class License:
     ## Web present of the license
     var web: String
 
-    func get_property_list() -> Array:
-        var properties: Array = super.get_property_list()
-        for property in properties:
-            if property["name"] == "text":
-                property["hint"] = PROPERTY_HINT_MULTILINE_TEXT
-            if property["name"] == "file":
-                property["hint"] = PROPERTY_HINT_FILE
-        return properties
+    func _get_property_list() -> Array[Dictionary]:
+        return [
+            {
+                "name": "text",
+                "type": TYPE_STRING,
+                "hint": PROPERTY_HINT_MULTILINE_TEXT,
+            },
+            {
+                "name": "file",
+                "type": TYPE_STRING,
+                "hint": PROPERTY_HINT_FILE,
+            }
+        ]
 
     ## Either returns the license text or loads the text from file or a message that the text could not be loaded.
     func get_license_text() -> String:
@@ -75,18 +80,25 @@ var web: String
 var paths: PackedStringArray = []
 var licenses: Array[License] = []
 
-func get_property_list() -> Array:
-    var properties: Array = super.get_property_list()
-    for property in properties:
-        if property["name"] == "paths":
-            property["hint"] = PROPERTY_HINT_FILE
-            # allow files too (inofficial)
-            property["hint_text"] = "*"
-        if property["name"] == "description":
-            property["hint"] = PROPERTY_HINT_MULTILINE_TEXT
-        if property["name"] == "licenses":
-            property["constructor"] = License.new
-    return properties
+func _get_property_list() -> Array:
+    return [
+        {
+            "name": "paths",
+            "type": TYPE_PACKED_STRING_ARRAY,
+            "hint": PROPERTY_HINT_FILE,
+            "hint_text": "*",
+        },
+        {
+            "name": "description",
+            "type": TYPE_STRING,
+            "hint": PROPERTY_HINT_MULTILINE_TEXT,
+        },
+        {
+            "name": "licenses",
+            "type": TYPE_ARRAY,
+            "constructor": License.new,
+        },
+    ]
 
 func serialize() -> Dictionary:
     var licenses: Array[Dictionary] = []

addons/licenses/internal/handler/object.gd

@@ -1,11 +1,14 @@
 extends "base.gd"
 
+const Utils := preload("../utils.gd")
+
 func _init(tree_: Tree, item_: TreeItem, value_: Variant, property_: Dictionary) -> void:
     super._init(tree_, item_, value_, property_)
     self.item.set_text(0, self.property["name"].capitalize())
     self.item.set_selectable(1, false)
-    for prop in self.value.get_property_list():
-        # ignore private variables and ignore non supported types
+
+    for prop in Utils.get_updated_property_list(self.value):
+        # ignore private variables and ignore non supported types and already added items
         if prop["name"].begins_with("_"):
             continue
         self.tree._add_item(self.item, self.value.get(prop["name"]), prop)

addons/licenses/internal/utils.gd

@@ -0,0 +1,10 @@
+static func get_updated_property_list(object: Object) -> Array:
+    var properties: Array = object.get_property_list()
+    var patched_properties: Array = object._get_property_list()
+    properties = properties.slice(0, properties.size() - patched_properties.size())
+    for p_prop_idx in range(patched_properties.size()):
+        for prop_idx in range(properties.size()):
+            if properties[prop_idx]["name"] == patched_properties[p_prop_idx]["name"]:
+                properties[prop_idx] = patched_properties[p_prop_idx]
+                break
+    return properties