summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRichard Hughes <richard@hughsie.com>2016-07-27 10:54:02 +0100
committerRichard Hughes <richard@hughsie.com>2016-07-27 10:54:02 +0100
commite5abe0fda12589d53612dae9aebe77f21ebfc2e9 (patch)
tree69fc328cf42bc780a1dfd0bbd6f36b9df8635765
parentc7fa3a21c0ca2719a397b191d742e88a45a265bd (diff)
downloadappstream-glib-e5abe0fda12589d53612dae9aebe77f21ebfc2e9.tar.gz
trivial: Remove AsReview.karma as it's implementation specific
-rw-r--r--libappstream-glib/as-review.c66
-rw-r--r--libappstream-glib/as-review.h3
-rw-r--r--libappstream-glib/as-self-test.c3
3 files changed, 1 insertions, 71 deletions
diff --git a/libappstream-glib/as-review.c b/libappstream-glib/as-review.c
index ed739da..9488ff6 100644
--- a/libappstream-glib/as-review.c
+++ b/libappstream-glib/as-review.c
@@ -46,7 +46,6 @@ typedef struct
gchar *summary;
gchar *description;
gchar *locale;
- gint karma;
gint priority;
gint rating;
gchar *version;
@@ -59,7 +58,6 @@ typedef struct
enum {
PROP_0,
PROP_ID,
- PROP_KARMA,
PROP_SUMMARY,
PROP_DESCRIPTION,
PROP_LOCALE,
@@ -103,9 +101,6 @@ as_review_get_property (GObject *object, guint prop_id,
AsReviewPrivate *priv = GET_PRIVATE (review);
switch (prop_id) {
- case PROP_KARMA:
- g_value_set_int (value, priv->karma);
- break;
case PROP_ID:
g_value_set_string (value, priv->id);
break;
@@ -149,9 +144,6 @@ as_review_set_property (GObject *object, guint prop_id,
AsReview *review = AS_REVIEW (object);
switch (prop_id) {
- case PROP_KARMA:
- as_review_set_karma (review, g_value_get_int (value));
- break;
case PROP_ID:
as_review_set_id (review, g_value_get_string (value));
break;
@@ -198,16 +190,6 @@ as_review_class_init (AsReviewClass *klass)
object_class->set_property = as_review_set_property;
/**
- * AsReview:karma:
- *
- * Since: 0.5.18
- **/
- pspec = g_param_spec_int ("karma", NULL, NULL,
- G_MININT, G_MAXINT, 0,
- G_PARAM_READWRITE | G_PARAM_CONSTRUCT);
- g_object_class_install_property (object_class, PROP_KARMA, pspec);
-
- /**
* AsReview:id:
*
* Since: 0.5.18
@@ -319,45 +301,6 @@ as_review_init (AsReview *review)
}
/**
- * as_review_get_karma:
- * @review: a #AsReview
- *
- * Gets the karma for the review, where positive numbers indicate
- * more positive feedback for the review.
- *
- * Returns: the karma value, or 0 for unset.
- *
- * Since: 0.5.18
- **/
-gint
-as_review_get_karma (AsReview *review)
-{
- AsReviewPrivate *priv = GET_PRIVATE (review);
- g_return_val_if_fail (AS_IS_REVIEW (review), 0);
- return priv->karma;
-}
-
-/**
- * as_review_set_karma:
- * @review: a #AsReview
- * @karma: a karma value
- *
- * Sets the karma for the review, where positive numbers indicate
- * more positive feedback for the review.
- *
- * Karma can be positive or negative, or 0 for unset.
- *
- * Since: 0.5.18
- **/
-void
-as_review_set_karma (AsReview *review, gint karma)
-{
- AsReviewPrivate *priv = GET_PRIVATE (review);
- g_return_if_fail (AS_IS_REVIEW (review));
- priv->karma = karma;
-}
-
-/**
* as_review_get_priority:
* @review: a #AsReview
*
@@ -838,8 +781,6 @@ as_review_equal (AsReview *review1, AsReview *review2)
/* check for equality */
if (!g_date_time_equal (priv1->date, priv2->date))
return FALSE;
- if (priv1->karma != priv2->karma)
- return FALSE;
if (priv1->priority != priv2->priority)
return FALSE;
if (priv1->rating != priv2->rating)
@@ -882,10 +823,6 @@ as_review_node_insert (AsReview *review, GNode *parent, AsNodeContext *ctx)
NULL);
if (priv->id != NULL)
as_node_add_attribute (n, "id", priv->id);
- if (priv->karma != 0) {
- g_autofree gchar *str = g_strdup_printf ("%i", priv->karma);
- as_node_add_attribute (n, "karma", str);
- }
if (priv->priority != 0) {
g_autofree gchar *str = g_strdup_printf ("%i", priv->priority);
as_node_insert (n, "priority", str,
@@ -963,9 +900,6 @@ as_review_node_parse (AsReview *review, GNode *node,
const gchar *tmp;
gint itmp;
- itmp = as_node_get_attribute_as_int (node, "karma");
- if (itmp != G_MAXINT)
- as_review_set_karma (review, itmp);
itmp = as_node_get_attribute_as_int (node, "rating");
if (itmp != G_MAXINT)
as_review_set_rating (review, itmp);
diff --git a/libappstream-glib/as-review.h b/libappstream-glib/as-review.h
index 35026bc..e19e9c8 100644
--- a/libappstream-glib/as-review.h
+++ b/libappstream-glib/as-review.h
@@ -69,7 +69,6 @@ typedef enum {
AsReview *as_review_new (void);
/* getters */
-gint as_review_get_karma (AsReview *review);
gint as_review_get_priority (AsReview *review);
const gchar *as_review_get_id (AsReview *review);
const gchar *as_review_get_summary (AsReview *review);
@@ -85,8 +84,6 @@ const gchar *as_review_get_metadata_item (AsReview *review,
const gchar *key);
/* setters */
-void as_review_set_karma (AsReview *review,
- gint karma);
void as_review_set_priority (AsReview *review,
gint priority);
void as_review_set_id (AsReview *review,
diff --git a/libappstream-glib/as-self-test.c b/libappstream-glib/as-self-test.c
index 87ec000..7ab1b33 100644
--- a/libappstream-glib/as-self-test.c
+++ b/libappstream-glib/as-self-test.c
@@ -1181,7 +1181,7 @@ as_test_review_func (void)
AsNode *root;
GString *xml;
const gchar *src =
- "<review date=\"2016-09-15\" rating=\"80\" karma=\"-1\" id=\"17\">\n"
+ "<review date=\"2016-09-15\" rating=\"80\" id=\"17\">\n"
"<priority>5</priority>\n"
"<summary>Hello world</summary>\n"
"<description><p>Mighty Fine</p></description>\n"
@@ -1212,7 +1212,6 @@ as_test_review_func (void)
as_node_unref (root);
/* verify */
- g_assert_cmpint (as_review_get_karma (review), ==, -1);
g_assert_cmpint (as_review_get_priority (review), ==, 5);
g_assert (as_review_get_date (review) != NULL);
g_assert_cmpstr (as_review_get_id (review), ==, "17");