vanitasvitae/Smack
1
0
Fork 0
mirror of https://github.com/vanitasvitae/Smack.git synced 2024-12-22 18:48:00 +01:00
Code Issues Releases Wiki Activity

[pubsub] Allow for character data before <item/>'s payload

Browse source 
Fixes SMACK-918.
This commit is contained in:
Florian Schmaus 2021-12-13 21:49:15 +01:00
parent e39adff40f
commit fc7fc10c69
2 changed files with 66 additions and 17 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
smack-extensions/src/main/java/org/jivesoftware/smackx/pubsub/provider/ItemProvider.java
View file

@ -50,24 +50,25 @@ public class ItemProvider extends ExtensionElementProvider<Item> {
String xmlns = parser.getNamespace(); String xmlns = parser.getNamespace();
ItemNamespace itemNamespace = ItemNamespace.fromXmlns(xmlns); ItemNamespace itemNamespace = ItemNamespace.fromXmlns(xmlns);
XmlPullParser.Event tag = parser.next(); XmlPullParser.TagEvent event = parser.nextTag();
switch (event) {
case START_ELEMENT:
String payloadElemName = parser.getName();
String payloadNS = parser.getNamespace();
if (tag == XmlPullParser.Event.END_ELEMENT) { final ExtensionElementProvider<ExtensionElement> extensionProvider = ProviderManager.getExtensionProvider(payloadElemName, payloadNS);
return new Item(itemNamespace, id, node); if (extensionProvider == null) {
} // TODO: Should we use StandardExtensionElement in this case? And probably remove SimplePayload all together.
else { CharSequence payloadText = PacketParserUtils.parseElement(parser, true);
String payloadElemName = parser.getName(); return new PayloadItem<>(itemNamespace, id, node, new SimplePayload(payloadText.toString()));
String payloadNS = parser.getNamespace(); }
else {
final ExtensionElementProvider<ExtensionElement> extensionProvider = ProviderManager.getExtensionProvider(payloadElemName, payloadNS); return new PayloadItem<>(itemNamespace, id, node, extensionProvider.parse(parser));
if (extensionProvider == null) { }
// TODO: Should we use StandardExtensionElement in this case? And probably remove SimplePayload all together. case END_ELEMENT:
CharSequence payloadText = PacketParserUtils.parseElement(parser, true); return new Item(itemNamespace, id, node);
return new PayloadItem<>(itemNamespace, id, node, new SimplePayload(payloadText.toString())); default:
} throw new AssertionError("unknown: " + event);
else {
return new PayloadItem<>(itemNamespace, id, node, extensionProvider.parse(parser));
}
} }
} }

48
smack-extensions/src/test/java/org/jivesoftware/smackx/pubsub/provider/ItemProviderTest.java Normal file
View file

@ -0,0 +1,48 @@
/**
*
* Copyright 2021 Florian Schmaus
*
* 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.provider;
import java.io.IOException;
import org.jivesoftware.smack.parsing.SmackParsingException;
import org.jivesoftware.smack.test.util.SmackTestUtil;
import org.jivesoftware.smack.xml.XmlPullParserException;
import org.junit.jupiter.params.ParameterizedTest;
import org.junit.jupiter.params.provider.EnumSource;
public class ItemProviderTest {
/**
* Check that {@link ItemProvider} is able to parse items which have whitespace before their Payload.
*
* @param parserKind the used parser Kind
* @throws XmlPullParserException if an XML pull parser exception occurs.
* @throws IOException if an IO exception occurs.
* @throws SmackParsingException if an Smack parsing exception occurs.
* @see <a href="https://igniterealtime.atlassian.net/jira/software/c/projects/SMACK/issues/SMACK-918">SMACK-918</a>
*/
@ParameterizedTest
@EnumSource(SmackTestUtil.XmlPullParserKind.class)
public void whitespaceBeforeItemPayload(SmackTestUtil.XmlPullParserKind parserKind) throws XmlPullParserException, IOException, SmackParsingException {
String item = "<item id='13a3710c-68c3-4da2-8484-d8d9c77af91e' xmlns='http://jabber.org/protocol/pubsub#event'>"
+ "\n <geoloc xmlns='http://jabber.org/protocol/geoloc' xml:lang='en'/>"
+ "</item>";
SmackTestUtil.parse(item, ItemProvider.class, parserKind);
}
}