godotengine
/
godot
mirror of https://github.com/godotengine/godot
1
0
Fork 0
Code

Clarify EditorImportPlugin.get_visible_name. 

According to the docs at
https://docs.godotengine.org/en/stable/tutorials/plugins/editor/import_plugins.html#the-editorimportplugin-class

> The get_visible_name() method is responsible for returning the name of
> the type it imports and it will be shown to the user in the Import dock.
> You should choose this name as a continuation to "Import as", e.g.
> "Import as Silly Material"

I've verified Godot's behavior reflects this, so the code examples
should reflect this.

Also document propagating save error in EditorImportPlugin.

It seems that the suggested code ignores any error from
`ResourceSaver.save`, but I think we should return it.

(cherry picked from commit 9676650f2f)
This commit is contained in:
Ryan Roden-Corrent 2021-02-09 08:30:27 -05:00 committed by Rémi Verschelde
parent cd2a996d9e
commit 08d556f677
No known key found for this signature in database
GPG Key ID: C3336907360768E1
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
doc/classes/EditorImportPlugin.xml
View File

@ -15,7 +15,7 @@
return "my.special.plugin"
func get_visible_name():
return "Special Mesh Importer"
return "Special Mesh"
func get_recognized_extensions():
return ["special", "spec"]
@ -44,8 +44,7 @@
# Fill the Mesh with data read in "file", left as an exercise to the reader
var filename = save_path + "." + get_save_extension()
ResourceSaver.save(filename, mesh)
return OK
return ResourceSaver.save(filename, mesh)
[/codeblock]
</description>
<tutorials>
@ -143,7 +142,7 @@
<return type="String">
</return>
<description>
Gets the name to display in the import window.
Gets the name to display in the import window. You should choose this name as a continuation to "Import as", e.g. "Import as Special Mesh".
</description>
</method>
<method name="import" qualifiers="virtual">