/**
 *
 * Copyright the original author or authors
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
 * You may obtain a copy of the License at
 *
 *     http://www.apache.org/licenses/LICENSE-2.0
 *
 * Unless required by applicable law or agreed to in writing, software
 * distributed under the License is distributed on an "AS IS" BASIS,
 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
 * See the License for the specific language governing permissions and
 * limitations under the License.
 */
package org.jivesoftware.smackx.pubsub;

import org.jxmpp.jid.Jid;

/**
 * Represents a request to subscribe to a node.
 *
 * @author Robin Collier
 */
public class SubscribeExtension extends NodeExtension {
    protected final Jid jid;

    public SubscribeExtension(Jid subscribeJid) {
        super(PubSubElementType.SUBSCRIBE);
        jid = subscribeJid;
    }

    public SubscribeExtension(Jid subscribeJid, String nodeId) {
        super(PubSubElementType.SUBSCRIBE, nodeId);
        jid = subscribeJid;
    }

    public Jid getJid() {
        return jid;
    }

    @Override
    public String toXML(org.jivesoftware.smack.packet.XmlEnvironment enclosingNamespace) {
        StringBuilder builder = new StringBuilder("<");
        builder.append(getElementName());

        if (getNode() != null) {
            builder.append(" node='");
            builder.append(getNode());
            builder.append('\'');
        }
        builder.append(" jid='");
        builder.append(getJid());
        builder.append("'/>");

        return builder.toString();
    }
}

@@ -647,16 +647,123 @@
 ubsub;%0A%0A
+import org.jivesoftware.smack.packet.XmlEnvironment;%0Aimport org.jivesoftware.smack.util.XmlStringBuilder;%0A%0A
 import o
@@ -1314,84 +1314,60 @@
 lic 
+Xml
 String
- toXML(org.jivesoftware.smack.packet.XmlEnvironment enclosingNamespace
+Builder toXML(XmlEnvironment xmlEnvironment
 ) %7B%0A
@@ -1374,16 +1374,19 @@
         
+Xml
 StringBu
@@ -1395,22 +1395,21 @@
 der 
-builder
+xml
  = new 
+Xml
 Stri
@@ -1422,65 +1422,39 @@
 der(
-%22%3C%22);%0A        builder.append(getElementName()
+this, xmlEnvironment
 );%0A
-%0A
         
 if (
@@ -1453,255 +1453,118 @@
     
-if (getNode() != null) %7B%0A            builder.append
+xml.optAttribute
 (%22
- 
 node
-='%22);%0A            builder.append(getNode());%0A            builder.append('%5C'');%0A        %7D%0A        builder.append
+%22, getNode());%0A        xml.attribute
 (%22
- 
 jid
-='%22);%0A        builder.append(getJid());%0A        builder.append(%22'/%3E%22
+%22, getJid());%0A        xml.closeEmptyElement(
 );%0A
-%0A
     
@@ -1578,26 +1578,11 @@
 urn 
-builder.toString()
+xml
 ;%0A  

Use XmlStringBuilder in PubSub's SubscribeExtension