[rygel] external: Simplify and fix translatable message
- From: Jens Georg <jensgeorg src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [rygel] external: Simplify and fix translatable message
- Date: Thu, 15 Aug 2013 10:28:00 +0000 (UTC)
commit be20dc1cc8bb7bf455b38117d8a6ebfab5ab4d25
Author: Jens Georg <jensg openismus com>
Date: Thu Aug 15 12:26:24 2013 +0200
external: Simplify and fix translatable message
https://bugzilla.gnome.org/show_bug.cgi?id=705899
.../rygel-dlna-profile.vala | 1 -
.../external/rygel-external-variant-util.vala | 33 ++++++++++++-------
2 files changed, 21 insertions(+), 13 deletions(-)
---
diff --git a/src/librygel-server/rygel-dlna-profile.vala b/src/librygel-core/rygel-dlna-profile.vala
similarity index 99%
rename from src/librygel-server/rygel-dlna-profile.vala
rename to src/librygel-core/rygel-dlna-profile.vala
index 5abb22e..20ec1c8 100644
--- a/src/librygel-server/rygel-dlna-profile.vala
+++ b/src/librygel-core/rygel-dlna-profile.vala
@@ -42,4 +42,3 @@ public class Rygel.DLNAProfile {
return a.name.ascii_casecmp (b.name);
}
}
-
diff --git a/src/plugins/external/rygel-external-variant-util.vala
b/src/plugins/external/rygel-external-variant-util.vala
index 1f564c1..3ef3d1c 100644
--- a/src/plugins/external/rygel-external-variant-util.vala
+++ b/src/plugins/external/rygel-external-variant-util.vala
@@ -19,18 +19,31 @@
*/
namespace Rygel.External {
- public static string get_mandatory_string_value
+ public static string MANDATORY_MISSING_MESSAGE =
+ N_("External provider %s did not provide mandatory property \"%s\"");
+
+ public static Variant? get_mandatory
(HashTable<string, Variant> props,
string key,
- string default,
string service_name) {
var value = props.lookup (key);
if (value == null) {
- warning (_("External provider %s did not provide mandatory " +
- "property \"%s\""),
- service_name,
- key);
+ warning (_(MANDATORY_MISSING_MESSAGE), service_name, key);
+
+ return null;
+ }
+ return value;
+ }
+
+ public static string get_mandatory_string_value
+ (HashTable<string, Variant> props,
+ string key,
+ string default,
+ string service_name) {
+ var value = get_mandatory (props, key, service_name);
+
+ if (value == null) {
return default;
}
@@ -42,13 +55,9 @@ namespace Rygel.External {
string key,
string[]? default,
string service_name) {
- var value = props.lookup (key);
- if (value == null) {
- warning (_("External provider %s did not provide mandatory " +
- "property \"%s\""),
- service_name,
- key);
+ var value = get_mandatory (props, key, service_name);
+ if (value == null) {
return default;
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]