Only add non-empty Role components to Edsf.Personas.

The EDS test "set-roles" was failing because some empty parameters were
getting added to Edsf.Personas, thus failing parameter equality checks.

Helps: bgo#662433 - AbstractFieldDetails.equal() is ambiguous about
checking parameters.

diff --git a/backends/eds/lib/edsf-persona.vala b/backends/eds/lib/edsf-persona.vala
index 382a347..0ec63f0 100644 (file)
--- a/backends/eds/lib/edsf-persona.vala
+++ b/backends/eds/lib/edsf-persona.vala
@@ -972,16 +972,16 @@ public class Edsf.Persona : Folks.Persona,
 
           default_role = new RoleFieldDetails (new_role);
 
-          if (org_unit != null)
+          if (org_unit != null && org_unit != "")
             default_role.set_parameter ("org_unit", org_unit);
 
-          if (office != null)
+          if (office != null && office != "")
             default_role.set_parameter ("office", office);
 
-          if (manager != null)
+          if (manager != null && manager != "")
             default_role.set_parameter ("manager", manager);
 
-          if (assistant != null)
+          if (assistant != null && manager != "")
             default_role.set_parameter ("assistant", assistant);
         }