<p dir="ltr">For twitter, we don't need XML support because the new a pi doesn't support it. This doesn't mean that there won't be another storage solution in the future but for now the first approach which would only affect twitter will do it.</p>

<p dir="ltr">For <a href="http://pump.io">pump.io</a> this is a different topic which will be addressed in it's own lib.</p>
<div class="gmail_quote">On Jun 20, 2013 9:47 AM, "Luis Angel Fernandez Fernandez" <<a href="mailto:laffdez@gmail.com">laffdez@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><br><div class="gmail_extra"><div class="gmail_quote">2013/6/20 Daniel Kreuter <span dir="ltr"><<a href="mailto:daniel.kreuter85@gmail.com" target="_blank">daniel.kreuter85@gmail.com</a>></span><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
First one:<br>
Since we now only have JSON, a simple renaming from readPostFromJson<br>
to readPost (as wel for the other methods) would do it.<br>
<br>
Second way:<br>
Introduce an abstract class or an interface which defines the methods<br>
and implement it with the appropriate type (f.e. one for json, one for<br>
xml and so on).<br>
This way we can construct the appropriate type based on the format<br>
used, and have one interface which rules them all.<br>
<br>
The benefit of both ways would be, that the code is more maintainable.<br>
No need to check for the format do call the correct version.<br>
<br>
What's your opinion?<br></blockquote><div><br></div><div>  Is/Will be there any need for keeping the XML support? If there is no need I'd vote for the first approach. If the XML support needs to be kept then I'd vote for the second approach.<br>

</div><div><br></div><div>  But I have a question. Since yo want to go into this refactoring thing :-). Would it be a good idea to use the work Martin Klapetek did with his akonadi plugin and the develop the <a href="http://pump.io" target="_blank">pump.io</a> support as an akonadi plugin?</div>

</div><div><br></div>-- <br>Linkedin profile (<a href="http://es.linkedin.com/in/lafdez" target="_blank">http://es.linkedin.com/in/lafdez</a>) <div>G+ profile (<a href="https://plus.google.com/u/0/115320207805121303027/about" target="_blank">https://plus.google.com/u/0/115320207805121303027/about</a>) </div>

<div>Twitter (@lafdez @_lafdez_) </div><div>Identi.ca (@lafdez)</div>
</div></div>
</blockquote></div>