Working patch is attached. Needs commenting.

CommentFileSizeAuthor
feedapi_opml_batch.patch6.15 KBmustafau

Comments

robertdouglass’s picture

robertdouglass commented

Very nice refactoring. This patch has more "goodness" in it than just the much needed batch api. First, you took the time to add checks to avoid E_ALL notices. Thank you. Second you extracted such functions as feedapi_feed_exists. Nice.

It would be nice to do some profiling on this code because it seems that opml importing takes much longer than it should.

mustafau’s picture

mustafau commented
Assigned: mustafau » Unassigned
Priority: Critical » Normal