PEPItem can only be extended if the new class is in the same package

Smack Smack Dev
1

I know this discussion has already taken place (http://community.igniterealtime.org/thread/36837) but nothing has come of it. Rather than post in an old thread I am creating a new one with the link to the old one. Are there any plans to implement the simple fix of changing the following two methods within PEPItem to public?

abstract String getNode();
abstract String getItemDetailsXML();

Yes the work around of placing the PEPItem the user wants to create in the same package (org.jivesoftware.smackx.packet) works but users shouldn’t have to do this to extend the class and use PEP via the Smack API.

2

The long term plan will be to rewrite PEP to use the Pubsub API.

Right now PEP defines some of the exact same xml stanzas as pubsub, which basically means you cannot use both together. PEP existed before pubsub, but it is in fact a subset of pubsub by definition and thus will at some point have to work together.

I will raise a task for this though, as it is a pretty straight forward fix.

SMACK-372

1 Like