diff options
author | Tobias Markmann <tm@ayena.de> | 2012-03-20 17:51:37 (GMT) |
---|---|---|
committer | Tobias Markmann <tm@ayena.de> | 2012-11-11 16:38:00 (GMT) |
commit | 6af3eb5576ae8475bcb1d545a563dcf059f9366f (patch) | |
tree | c3a9fad97f496fbdefef41d850609fe9cbf877ec /Swiften/Serializer | |
parent | 684559ff499077e33d3d5e4b9ecb39afa240d067 (diff) | |
download | swift-contrib-6af3eb5576ae8475bcb1d545a563dcf059f9366f.zip swift-contrib-6af3eb5576ae8475bcb1d545a563dcf059f9366f.tar.bz2 |
Serializer cleanup. More make_shared usage.
Diffstat (limited to 'Swiften/Serializer')
-rw-r--r-- | Swiften/Serializer/PayloadSerializers/VCardSerializer.cpp | 120 |
1 files changed, 44 insertions, 76 deletions
diff --git a/Swiften/Serializer/PayloadSerializers/VCardSerializer.cpp b/Swiften/Serializer/PayloadSerializers/VCardSerializer.cpp index 2676a02..6c6201b 100644 --- a/Swiften/Serializer/PayloadSerializers/VCardSerializer.cpp +++ b/Swiften/Serializer/PayloadSerializers/VCardSerializer.cpp @@ -18,61 +18,39 @@ namespace Swift { -boost::shared_ptr<XMLElement> createTextElement(const std::string& elementName, const std::string& text) { - boost::shared_ptr<XMLElement> element = boost::make_shared<XMLElement>(elementName); - element->addNode(boost::make_shared<XMLTextNode>(text)); - return element; -} - VCardSerializer::VCardSerializer() : GenericPayloadSerializer<VCard>() { } std::string VCardSerializer::serializePayload(boost::shared_ptr<VCard> vcard) const { XMLElement queryElement("vCard", "vcard-temp"); if (!vcard->getVersion().empty()) { - boost::shared_ptr<XMLElement> versionElement(new XMLElement("VERSION")); - versionElement->addNode(boost::make_shared<XMLTextNode>(vcard->getVersion())); - queryElement.addNode(versionElement); + queryElement.addNode(boost::make_shared<XMLElement>("VERSION", "", vcard->getVersion())); } if (!vcard->getFullName().empty()) { - boost::shared_ptr<XMLElement> fullNameElement(new XMLElement("FN")); - fullNameElement->addNode(boost::make_shared<XMLTextNode>(vcard->getFullName())); - queryElement.addNode(fullNameElement); + queryElement.addNode(boost::make_shared<XMLElement>("FN", "", vcard->getFullName())); } if (!vcard->getGivenName().empty() || !vcard->getFamilyName().empty() || !vcard->getMiddleName().empty() || !vcard->getPrefix().empty() || !vcard->getSuffix().empty()) { boost::shared_ptr<XMLElement> nameElement(new XMLElement("N")); if (!vcard->getFamilyName().empty()) { - boost::shared_ptr<XMLElement> familyNameElement(new XMLElement("FAMILY")); - familyNameElement->addNode(boost::make_shared<XMLTextNode>(vcard->getFamilyName())); - nameElement->addNode(familyNameElement); + nameElement->addNode(boost::make_shared<XMLElement>("FAMILY", "", vcard->getFamilyName())); } if (!vcard->getGivenName().empty()) { - boost::shared_ptr<XMLElement> givenNameElement(new XMLElement("GIVEN")); - givenNameElement->addNode(boost::make_shared<XMLTextNode>(vcard->getGivenName())); - nameElement->addNode(givenNameElement); + nameElement->addNode(boost::make_shared<XMLElement>("GIVEN", "", vcard->getGivenName())); } if (!vcard->getMiddleName().empty()) { - boost::shared_ptr<XMLElement> middleNameElement(new XMLElement("MIDDLE")); - middleNameElement->addNode(boost::make_shared<XMLTextNode>(vcard->getMiddleName())); - nameElement->addNode(middleNameElement); + nameElement->addNode(boost::make_shared<XMLElement>("MIDDLE", "", vcard->getMiddleName())); } if (!vcard->getPrefix().empty()) { - boost::shared_ptr<XMLElement> prefixElement(new XMLElement("PREFIX")); - prefixElement->addNode(boost::make_shared<XMLTextNode>(vcard->getPrefix())); - nameElement->addNode(prefixElement); + nameElement->addNode(boost::make_shared<XMLElement>("PREFIX", "", vcard->getPrefix())); } if (!vcard->getSuffix().empty()) { - boost::shared_ptr<XMLElement> suffixElement(new XMLElement("SUFFIX")); - suffixElement->addNode(boost::make_shared<XMLTextNode>(vcard->getSuffix())); - nameElement->addNode(suffixElement); + nameElement->addNode(boost::make_shared<XMLElement>("SUFFIX", "", vcard->getSuffix())); } queryElement.addNode(nameElement); } foreach(const VCard::EMailAddress& emailAddress, vcard->getEMailAddresses()) { boost::shared_ptr<XMLElement> emailElement(new XMLElement("EMAIL")); - boost::shared_ptr<XMLElement> userIDElement(new XMLElement("USERID")); - userIDElement->addNode(boost::make_shared<XMLTextNode>(emailAddress.address)); - emailElement->addNode(userIDElement); + emailElement->addNode(boost::make_shared<XMLElement>("USERID", "", emailAddress.address)); if (emailAddress.isHome) { emailElement->addNode(boost::make_shared<XMLElement>("HOME")); } @@ -91,99 +69,89 @@ std::string VCardSerializer::serializePayload(boost::shared_ptr<VCard> vcard) c queryElement.addNode(emailElement); } if (!vcard->getNickname().empty()) { - boost::shared_ptr<XMLElement> nickElement(new XMLElement("NICKNAME")); - nickElement->addNode(boost::make_shared<XMLTextNode>(vcard->getNickname())); - queryElement.addNode(nickElement); + queryElement.addNode(boost::make_shared<XMLElement>("NICKNAME", "", vcard->getNickname())); } if (!vcard->getPhoto().empty() || !vcard->getPhotoType().empty()) { XMLElement::ref photoElement(new XMLElement("PHOTO")); if (!vcard->getPhotoType().empty()) { - XMLElement::ref typeElement(new XMLElement("TYPE")); - typeElement->addNode(XMLTextNode::ref(new XMLTextNode(vcard->getPhotoType()))); - photoElement->addNode(typeElement); + photoElement->addNode(boost::make_shared<XMLElement>("TYPE", "", vcard->getPhotoType())); } if (!vcard->getPhoto().empty()) { - XMLElement::ref binvalElement(new XMLElement("BINVAL")); - binvalElement->addNode(XMLTextNode::ref(new XMLTextNode(Base64::encode(vcard->getPhoto())))); - photoElement->addNode(binvalElement); + photoElement->addNode(boost::make_shared<XMLElement>("BINVAL", "", Base64::encode(vcard->getPhoto()))); } queryElement.addNode(photoElement); } if (!vcard->getBirthday().is_not_a_date_time()) { - XMLElement::ref bdayElement(new XMLElement("BDAY")); - bdayElement->addNode(boost::shared_ptr<XMLTextNode>(new XMLTextNode(dateTimeToString(vcard->getBirthday())))); - queryElement.addNode(bdayElement); + queryElement.addNode(boost::make_shared<XMLElement>("BDAY", "", dateTimeToString(vcard->getBirthday()))); } foreach(const VCard::Telephone& telephone, vcard->getTelephones()) { boost::shared_ptr<XMLElement> telElement(new XMLElement("TEL")); - boost::shared_ptr<XMLElement> numberElement(new XMLElement("NUMBER")); - numberElement->addNode(boost::shared_ptr<XMLTextNode>(new XMLTextNode(telephone.number))); - telElement->addNode(numberElement); + telElement->addNode(boost::make_shared<XMLElement>("NUMBER", "", telephone.number)); if (telephone.isHome) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("HOME"))); + telElement->addNode(boost::make_shared<XMLElement>("HOME")); } if (telephone.isWork) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("WORK"))); + telElement->addNode(boost::make_shared<XMLElement>("WORK")); } if (telephone.isVoice) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("VOICE"))); + telElement->addNode(boost::make_shared<XMLElement>("VOICE")); } if (telephone.isFax) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("FAX"))); + telElement->addNode(boost::make_shared<XMLElement>("FAX")); } if (telephone.isPager) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("PAGER"))); + telElement->addNode(boost::make_shared<XMLElement>("PAGER")); } if (telephone.isMSG) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("MSG"))); + telElement->addNode(boost::make_shared<XMLElement>("MSG")); } if (telephone.isCell) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("CELL"))); + telElement->addNode(boost::make_shared<XMLElement>("CELL")); } if (telephone.isVideo) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("VIDEO"))); + telElement->addNode(boost::make_shared<XMLElement>("VIDEO")); } if (telephone.isBBS) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("BBS"))); + telElement->addNode(boost::make_shared<XMLElement>("BBS")); } if (telephone.isModem) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("MODEM"))); + telElement->addNode(boost::make_shared<XMLElement>("MODEM")); } if (telephone.isISDN) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("ISDN"))); + telElement->addNode(boost::make_shared<XMLElement>("ISDN")); } if (telephone.isPCS) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("PCS"))); + telElement->addNode(boost::make_shared<XMLElement>("PCS")); } if (telephone.isPreferred) { - telElement->addNode(boost::shared_ptr<XMLElement>(new XMLElement("PREF"))); + telElement->addNode(boost::make_shared<XMLElement>("PREF")); } queryElement.addNode(telElement); } foreach(const VCard::Address& address, vcard->getAddresses()) { - boost::shared_ptr<XMLElement> adrElement(new XMLElement("ADR")); + boost::shared_ptr<XMLElement> adrElement = boost::make_shared<XMLElement>("ADR"); if (!address.POBox.empty()) { - adrElement->addNode(createTextElement("POBOX", address.POBox)); + adrElement->addNode(boost::make_shared<XMLElement>("POBOX", "", address.POBox)); } if (!address.addressExtension.empty()) { - adrElement->addNode(createTextElement("EXTADD", address.addressExtension)); + adrElement->addNode(boost::make_shared<XMLElement>("EXTADD", "", address.addressExtension)); } if (!address.street.empty()) { - adrElement->addNode(createTextElement("STREET", address.street)); + adrElement->addNode(boost::make_shared<XMLElement>("STREET", "", address.street)); } if (!address.locality.empty()) { - adrElement->addNode(createTextElement("LOCALITY", address.locality)); + adrElement->addNode(boost::make_shared<XMLElement>("LOCALITY", "", address.locality)); } if (!address.region.empty()) { - adrElement->addNode(createTextElement("REGION", address.region)); + adrElement->addNode(boost::make_shared<XMLElement>("REGION", "", address.region)); } if (!address.postalCode.empty()) { - adrElement->addNode(createTextElement("PCODE", address.postalCode)); + adrElement->addNode(boost::make_shared<XMLElement>("PCODE", "", address.postalCode)); } if (!address.country.empty()) { - adrElement->addNode(createTextElement("CTRY", address.country)); + adrElement->addNode(boost::make_shared<XMLElement>("CTRY", "", address.country)); } if (address.isHome) { @@ -211,10 +179,10 @@ std::string VCardSerializer::serializePayload(boost::shared_ptr<VCard> vcard) c } foreach(const VCard::AddressLabel& addressLabel, vcard->getAddressLabels()) { - boost::shared_ptr<XMLElement> labelElement(new XMLElement("LABEL")); + boost::shared_ptr<XMLElement> labelElement = boost::make_shared<XMLElement>("LABEL"); foreach(const std::string& line, addressLabel.lines) { - labelElement->addNode(createTextElement("LINE", line)); + labelElement->addNode(boost::make_shared<XMLElement>("LINE", "", line)); } if (addressLabel.isHome) { @@ -242,36 +210,36 @@ std::string VCardSerializer::serializePayload(boost::shared_ptr<VCard> vcard) c } foreach(const JID& jid, vcard->getJIDs()) { - queryElement.addNode(createTextElement("JID", jid.toString())); + queryElement.addNode(boost::make_shared<XMLElement>("JID", "", jid.toString())); } if (!vcard->getDescription().empty()) { - queryElement.addNode(createTextElement("DESC", vcard->getDescription())); + queryElement.addNode(boost::make_shared<XMLElement>("DESC", "", vcard->getDescription())); } foreach(const VCard::Organization& org, vcard->getOrganizations()) { - boost::shared_ptr<XMLElement> orgElement(new XMLElement("ORG")); + boost::shared_ptr<XMLElement> orgElement = boost::make_shared<XMLElement>("ORG"); if (!org.name.empty()) { - orgElement->addNode(createTextElement("ORGNAME", org.name)); + orgElement->addNode(boost::make_shared<XMLElement>("ORGNAME", "", org.name)); } if (!org.units.empty()) { foreach(const std::string& unit, org.units) { - orgElement->addNode(createTextElement("ORGUNIT", unit)); + orgElement->addNode(boost::make_shared<XMLElement>("ORGUNIT", "", unit)); } } queryElement.addNode(orgElement); } foreach(const std::string& title, vcard->getTitles()) { - queryElement.addNode(createTextElement("TITLE", title)); + queryElement.addNode(boost::make_shared<XMLElement>("TITLE", "", title)); } foreach(const std::string& role, vcard->getRoles()) { - queryElement.addNode(createTextElement("ROLE", role)); + queryElement.addNode(boost::make_shared<XMLElement>("ROLE", "", role)); } foreach(const std::string& url, vcard->getURLs()) { - queryElement.addNode(createTextElement("URL", url)); + queryElement.addNode(boost::make_shared<XMLElement>("URL", "", url)); } if (!vcard->getUnknownContent().empty()) { |