From 5810ebb18f1b373b6045e87781d0ea310a89f2ad Mon Sep 17 00:00:00 2001 From: Sam Dodrill Date: Sun, 24 Nov 2013 14:11:00 -0800 Subject: [PATCH] modules/code/m_metadata: Changed to conform to IRCv3 command spec --- modules/core/m_metadata.c | 13 +++++++++---- src/channel.c | 4 ++-- src/client.c | 4 ++-- src/s_serv.c | 4 ++-- 4 files changed, 15 insertions(+), 10 deletions(-) diff --git a/modules/core/m_metadata.c b/modules/core/m_metadata.c index 77fe0fe..eba59ca 100644 --- a/modules/core/m_metadata.c +++ b/modules/core/m_metadata.c @@ -30,6 +30,11 @@ mapi_clist_av1 metadata_clist[] = { DECLARE_MODULE_AV1(metadata, NULL, NULL, metadata_clist, NULL, NULL, "$Revision$"); +/* + * Follows the specification for IRCv3 METADATA + * http://ircv3.org/specification/metadata-3.2 + */ + static int me_metadata(struct Client *client_p, struct Client *source_p, int parc, const char *parv[]) { @@ -40,9 +45,9 @@ me_metadata(struct Client *client_p, struct Client *source_p, int parc, const ch if((chptr = find_channel(parv[2])) == NULL) return 0; - if(!strcmp(parv[1], "ADD") && parv[4] != NULL) + if(!strcmp(parv[1], "SET") && parv[4] != NULL) channel_metadata_add(chptr, parv[3], parv[4], 0); - if(!strcmp(parv[1], "DELETE") && parv[3] != NULL) + if(!strcmp(parv[1], "CLEAR") && parv[3] != NULL) channel_metadata_delete(chptr, parv[3], 0); } @@ -56,9 +61,9 @@ me_metadata(struct Client *client_p, struct Client *source_p, int parc, const ch if(!target_p->user) return 0; - if(!strcmp(parv[1], "ADD") && parv[4] != NULL) + if(!strcmp(parv[1], "SET") && parv[4] != NULL) user_metadata_add(target_p, parv[3], parv[4], 0); - if(!strcmp(parv[1], "DELETE") && parv[3] != NULL) + if(!strcmp(parv[1], "CLEAR") && parv[3] != NULL) user_metadata_delete(target_p, parv[3], 0); } return 0; diff --git a/src/channel.c b/src/channel.c index 49bca01..a01ede0 100644 --- a/src/channel.c +++ b/src/channel.c @@ -2110,7 +2110,7 @@ channel_metadata_add(struct Channel *target, const char *name, const char *value irc_dictionary_add(target->metadata, md->name, md); if(propegate) - sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA ADD %s %s :%s", + sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA SET %s %s :%s", target->chname, name, value); return md; @@ -2163,7 +2163,7 @@ channel_metadata_delete(struct Channel *target, const char *name, int propegate) rb_free(md); if(propegate) - sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA DELETE %s %s", + sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA CLEAR %s %s", target->chname, name); } diff --git a/src/client.c b/src/client.c index f763b27..b4dd58f 100644 --- a/src/client.c +++ b/src/client.c @@ -1955,7 +1955,7 @@ user_metadata_add(struct Client *target, const char *name, const char *value, in irc_dictionary_add(target->user->metadata, md->name, md); if(propegate) - sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA ADD %s %s :%s", + sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA SET %s %s :%s", target->id, name, value); return md; @@ -1983,7 +1983,7 @@ user_metadata_delete(struct Client *target, const char *name, int propegate) rb_free(md); if(propegate) - sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA DELETE %s %s", + sendto_match_servs(&me, "*", CAP_ENCAP, NOCAPS, "ENCAP * METADATA CLEAR %s %s", target->id, name); } diff --git a/src/s_serv.c b/src/s_serv.c index 46fbcf0..e4576c6 100644 --- a/src/s_serv.c +++ b/src/s_serv.c @@ -598,7 +598,7 @@ burst_TS6(struct Client *client_p) DICTIONARY_FOREACH(md, &iter, target_p->user->metadata) { - sendto_one(client_p, ":%s ENCAP * METADATA ADD %s %s :%s", + sendto_one(client_p, ":%s ENCAP * METADATA SET %s %s :%s", use_id(&me), use_id(target_p), md->name, md->value); } @@ -666,7 +666,7 @@ burst_TS6(struct Client *client_p) { /* don't bother bursting +J metadata */ if(!(md->name[0] == 'K')) - sendto_one(client_p, ":%s ENCAP * METADATA ADD %s %s :%s", + sendto_one(client_p, ":%s ENCAP * METADATA SET %s %s :%s", use_id(&me), chptr->chname, md->name, md->value); }