[libgames-support] scores: warn when a dialog is created without any scores
- From: Michael Catanzaro <mcatanzaro src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [libgames-support] scores: warn when a dialog is created without any scores
- Date: Sun, 22 Feb 2015 18:23:32 +0000 (UTC)
commit 34e2da1fa4eac6a5e60ed6700e6716b3940d5436
Author: Michael Catanzaro <mcatanzaro gnome org>
Date: Sun Feb 22 11:22:54 2015 -0600
scores: warn when a dialog is created without any scores
scores/dialog.vala | 33 +++++++++++++++++----------------
1 files changed, 17 insertions(+), 16 deletions(-)
---
diff --git a/scores/dialog.vala b/scores/dialog.vala
index 6316edc..dd810de 100644
--- a/scores/dialog.vala
+++ b/scores/dialog.vala
@@ -186,29 +186,30 @@ private class Dialog : Gtk.Dialog
if (combo == null)
return;
+ if (scores.has_scores ())
+ {
+ warning ("A GamesScoresDialog was created but no scores exist yet. " +
+ "Games should not allow this dialog to be created before scores have been added. " +
+ "Use games_scores_context_has_scores() to determine if this dialog should be
available.");
+ return;
+ }
+
var categories = scores.get_categories ();
categories.foreach ((x) => combo.append (x.key, x.name));
- if (categories.length() > 0)
- {
- if (scores_active_category == null)
- combo.active_id = categories.nth_data (0).key;
- else
- combo.active_id = scores_active_category.key;
+ if (scores_active_category == null)
+ combo.active_id = categories.nth_data (0).key;
+ else
+ combo.active_id = scores_active_category.key;
- if (active_category == null)
- {
- active_category = new Category (categories.nth_data (0).key, categories.nth_data (0).name);
- }
- else
- {
- active_category.key = categories.nth_data (0).key;
- active_category.name = categories.nth_data (0).name;
- }
+ if (active_category == null)
+ {
+ active_category = new Category (categories.nth_data (0).key, categories.nth_data (0).name);
}
else
{
- active_category = null;
+ active_category.key = categories.nth_data (0).key;
+ active_category.name = categories.nth_data (0).name;
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]