move OPML import functions with the rest of the feed funcs
this is to make unit test easier
View Articleallow duplicate feed imports by abusing the folder name
we use the folder name in the title since it's the only way to import duplicates that could logically occur under different folders we also use the full folder path instead of just the last element
View Articlesilence coverage warnings for sanity checks
those checks should normally never be ran by the test suite, so they don't merit coverage...
View Articleadd simple opml integration tests
we do only basic tests here, complete OPML tests are in test_feeds we do not, for example, test folder exports, which do not work
View Articleproperly encode From headers
the default From header encoding is dumb: it encodes the *whole* string, including the address itself, which breaks a bunch of email clients, including gmail, notmuch and mutt. instead, we encode the...
View Articlereturn bytes properly in fetch
this fixes double-encoding issues detected in the previous test. our documentation was *saying* we were returning bytes from fetch() but, actually, we were returning a string. it *seems* that...
View ArticleMerge tag '0.8.0' into betamax
fixed encoding issues, folder support in OPML, bash completion
View Articlerefactor feeds to remove global
we now use a class static session that can be used from plugins this slightly simplifies the implementation of plugins as we do not need to load the module at a specific point to get the right...
View Articleadd requests-file dependency
get rid of our custom wrapper code around requests to handle file:// resources: that is handled by a third-party library and removes about 40 lines of code we didn't need this makes session...
View Articleuse getter/setters for class session
this makes the code cleaner and more idiomatic this does require plugins to use the private member to access the session, especially for the test suite, but there are ways of avoiding that for...
View Articlereorder feed class hierarchy
the feed fetch functionality does not need to be in the "storage" hierarchy...
View Articlemove opml import in the FeedStorage abstract class
this makes more sense than having it as a standalone function. also, it is not storage specific, so let's make a API-facing class that will expose more generic functions.
View Articlesplit --config and --database settings, using xdg module
the xdg module saves us some code and also extends configuration to support global config files like /etc or /var/cache (in theory - in practice, the freedesktop standards only use ~/.cache). using...
View Articleinitialize FeedFetcher session singleton properly
without this, the wayback plugin would fail because it FeedFetcher._session is None. doing this in the constructor makes the accessor faster as well, as we don't need to do another check. this should...
View Article
