aboutsummaryrefslogtreecommitdiff
path: root/game/src/Credits/Credits.gd
diff options
context:
space:
mode:
Diffstat (limited to 'game/src/Credits/Credits.gd')
-rw-r--r--game/src/Credits/Credits.gd96
1 files changed, 54 insertions, 42 deletions
diff --git a/game/src/Credits/Credits.gd b/game/src/Credits/Credits.gd
index 902207b..0d13424 100644
--- a/game/src/Credits/Credits.gd
+++ b/game/src/Credits/Credits.gd
@@ -32,72 +32,85 @@ var label_settings_personel : LabelSettings
@export
var credits_list: VBoxContainer
+const title_key : String = "TITLE"
+
# REQUIREMENTS:
# * 1.5 Credits Menu
# * SS-17
# REQUIREMENTS
# * FS-4
-func load_credit_file(path):
- var roles = {}
+func _load_credit_file(path : String) -> Dictionary:
+ var roles := {}
var core_credits = FileAccess.open(path, FileAccess.READ)
+ if core_credits == null:
+ push_error("Failed to open credits file %s (error code %d)" % [path, FileAccess.get_open_error()])
+ return roles
while not core_credits.eof_reached():
- var line = core_credits.get_csv_line()
+ var line := core_credits.get_csv_line()
+ var role := line[0].strip_edges().to_upper()
# If the line does not have an identifiable role or is empty then skip it
- if line[0].is_empty():
+ if role.is_empty() or line.size() < 2:
+ if not (role.is_empty() and line.size() < 2):
+ push_warning("Incorrectly formatted credit line %s in %s" % [line, path])
continue
- var role := line[0].to_upper()
- var person : String = line[1] # Cannot infer with := as line[1] does not have a set type
+ var person := line[1].strip_edges()
- if role not in roles.keys():
- roles[role] = []
- roles[role].push_back(person)
- else:
- roles[role].push_back(person)
+ if person.is_empty():
+ push_warning("Incorrectly formatted credit line %s in %s" % [line, path])
+ continue
+ if line.size() > 2:
+ push_warning("Extra entries ignored in credit line %s in %s" % [line, path])
+ if role not in roles:
+ roles[role] = [person]
+ else:
+ if person in roles[role]:
+ push_warning("Duplicate person %s for role %s in %s" % [person, role, path])
+ else:
+ roles[role].push_back(person)
+ if title_key in roles:
+ if roles[title_key].size() > 1:
+ push_warning("More than one %s: %s in %s" % [title_key, roles[title_key], path])
+ roles[title_key] = [roles[title_key][0]]
+ else:
+ push_warning("Credits file %s missing %s" % [path, title_key])
+ for role_list in roles.values():
+ role_list.sort_custom(func(a : String, b : String) -> bool: return a.naturalnocasecmp_to(b) < 0)
return roles
+func _add_label(node : Node, text : String, settings : LabelSettings) -> void:
+ var label := Label.new()
+ label.name = 'Label' + text
+ label.text = text
+ label.horizontal_alignment = HORIZONTAL_ALIGNMENT_CENTER
+ label.label_settings = settings
+ node.add_child(label)
+
# REQUIREMENTS:
# * UI-34, UI-35
-func make_project_credits(project):
+func _add_project_credits(project : Dictionary) -> void:
var project_credits_list = VBoxContainer.new()
- var project_credits_label = Label.new()
-
-
- # Spartan has some suggestions here but for now its good enough. Refer
- # to PR 16 resolved comments for further details
- project_credits_list.name = 'Credits' + project['TITLE'][0]
- project_credits_label.name = 'Label' + project['TITLE'][0]
- project_credits_label.text = project['TITLE'][0]
- project_credits_label.horizontal_alignment = HORIZONTAL_ALIGNMENT_CENTER
- project_credits_label.label_settings = label_settings_project
+ project_credits_list.name = 'Credits'
+ if title_key in project:
+ var title : String = project[title_key][0]
+ project_credits_list.name += title
+ _add_label(project_credits_list, title, label_settings_project)
+ project_credits_list.add_child(HSeparator.new())
- for role in project.keys():
- if role == 'TITLE':
+ for role in project:
+ if role == title_key:
continue
var role_parent = VBoxContainer.new()
- var role_label = Label.new()
-
- role_parent.name = 'Role' + role
- role_label.name = 'Label' + role
- role_label.text = role
- role_label.horizontal_alignment = HORIZONTAL_ALIGNMENT_CENTER
- role_label.label_settings = label_settings_role
for person in project[role]:
- var person_label = Label.new()
-
- person_label.name = 'Label' + person
- person_label.text = person
- person_label.horizontal_alignment = HORIZONTAL_ALIGNMENT_CENTER
- person_label.label_settings = label_settings_personel
- role_parent.add_child(person_label)
+ _add_label(role_parent, person, label_settings_personel)
- project_credits_list.add_child(role_label)
+ _add_label(project_credits_list, role, label_settings_role)
project_credits_list.add_child(role_parent)
project_credits_list.add_child(HSeparator.new())
@@ -106,11 +119,10 @@ func make_project_credits(project):
# REQUIREMENTS:
# * SS-17
func _ready():
- var core_credits = load_credit_file(core_credits_path)
- make_project_credits(core_credits)
+ _add_project_credits(_load_credit_file(core_credits_path))
# REQUIREMENTS:
# * UI-38
# * UIFUN-37
-func _on_back_button_pressed():
+func _on_back_button_pressed() -> void:
back_button_pressed.emit()