Commit Graph

83 Commits

Author SHA1 Message Date
Duke a3882284e0 test conservatives feed 2014-01-18 14:35:00 -02:00
jimt 269db380a3 Merge pull request #27 from haarts/master
Resolve uniqueness confusion
2013-12-06 02:08:15 -08:00
Harm Aarts 49e91ffeee Adds test for the new items 2013-12-05 16:29:55 +01:00
Harm Aarts 5a2d6dbcdc Revert "Revert "Adds new test data""
This reverts commit 79d418e00a.
2013-12-05 16:24:48 +01:00
Harm Aarts 0aaac62e64 Add check if the handler exists. 2013-12-05 16:23:32 +01:00
Harm Aarts b4a80d771a Adds a fallback ID function. 2013-12-05 16:18:13 +01:00
Harm Aarts 6b6086e389 Adds Key() 2013-12-05 16:04:00 +01:00
Harm Aarts 3b336dc54b Adds the database for checking new channels/items.
In this iteration the key passed to the database is the Title which is
obviously silly.
I'm still looking for a configurable way of generating the unique key.
2013-12-05 15:31:49 +01:00
Harm Aarts 90c93b8fa4 Adds a database.
This is a really simple map. Calls to it just check if the key exists
and return a bool. In case of a false it adds the key.
The key is just a string which might or might not be sufficient.
2013-12-05 15:30:26 +01:00
Harm Aarts bfbe2dd3be Fixes the tests
Reference the item directly through channel as the notification
functionality has been removed.
2013-12-05 14:22:03 +01:00
Harm Aarts e345751986 Removes new Item notification 2013-12-05 14:21:32 +01:00
Harm Aarts d75037c0ab Removes notification of new channels
This is going to be moved to the stateful Feed, this is yet to be
implemented.
2013-12-05 14:10:22 +01:00
Harm Aarts 73442f806a Makes atom replace channels and items, thus making it stateless 2013-12-05 14:08:41 +01:00
Harm Aarts 6ef84d35e2 Makes rss replace channels and items, thus making it stateless 2013-12-05 14:05:24 +01:00
Harm Aarts 4fa6c97010 Makes days map a global lookup table 2013-12-05 13:53:57 +01:00
Harm Aarts 79d418e00a Revert "Adds new test data"
This reverts commit 997070656f.
2013-12-05 12:56:02 +01:00
Harm Aarts 902e928863 Revert "Adds test"
This reverts commit b311fe3f8d.
2013-12-05 12:56:00 +01:00
Harm Aarts cc25852f30 Revert "Implements check if item is already present"
This reverts commit d79101645d.
2013-12-05 12:55:59 +01:00
jimt b4cf92a0fc Merge pull request #25 from haarts/master
Prevent re-adding Atom entries
2013-12-02 08:27:13 -08:00
Harm Aarts d79101645d Implements check if item is already present 2013-12-02 14:25:45 +01:00
Harm Aarts b311fe3f8d Adds test 2013-12-02 14:25:13 +01:00
Harm Aarts 997070656f Adds new test data 2013-12-02 14:24:57 +01:00
Jim Teeuwen 7b1a94122f Runs go fmt on all code. 2013-11-28 10:23:49 +01:00
Jim Teeuwen 29bd83be41 Merge Pull Request and issue #24
Turns Item.Guid field into a string pointer, so it may properly
be set to nil when applicable. Adjusts remaining code and tests
to reflect this change.
2013-11-28 10:22:09 +01:00
jimt 3537b8643d Merge pull request #23 from MJDSys/FixFeedParsingForxmlx
Fix Atom/RSS feed parsing due to changes in jteeuwen/go-pkg-xmlx.
2013-10-26 02:12:37 -07:00
Matthew Dawson 66eea2e6af Fix RSS feed parsing due to changes in go-pkg-xmlx.
Due to the PR jteeuwen/go-pkg-xmlx#16, when using SelectNodes there is no
longer any hidden recursion.  Due to RSS's structure, there is a root document
node.  These two pieces break the RSS parsing.  Fix by first selecting the
root document node, and then selecting the channels for parsing.
2013-10-25 23:26:15 -04:00
Matthew Dawson 20d050f908 Fix Atom/RSS feed parsing due to changes in jteeuwen/go-pkg-xmlx.
Due to recent changes involving how values are dealt with in xmlx, update
the RSS/Atom parsing.  Instead of using the Value property of an xmlx Node,
use the new GetValue function offered in PR jteeuwen/go-pkg-xmlx#15.
2013-09-30 01:23:51 -04:00
jimt c173d50291 Merge pull request #20 from MJDSys/RemoveHaveItem
Remove the haveItem check for RSS and Atom
2013-07-07 15:09:38 -07:00
jimt 5b836b89eb Merge pull request #19 from MJDSys/FixAuthor
Fix parsing authors in RSS feeds.
2013-07-07 15:07:58 -07:00
Matthew Dawson 9123b6bc67 Remove the haveItem check for RSS and Atom
The haveItem check for RSS and Atom causes the feeds to act unexpectedly.
For RSS, the checked tags don't necessarily have to be unique.  For atom,
it is allowed to have duplicate items (including duplicated ids) in one feed,
so this shouldn't be stopped either.
2013-07-07 12:03:24 -04:00
Matthew Dawson 41ad7d65b5 Fix parsing authors in RSS feeds.
In RSS feeds, the author of an item was always overwritten by a non-standard
creator tag.  Change this so that creator is only used when it actually
appears.  Otherwise use the previous value of Author, whatever that is.
2013-07-07 12:02:43 -04:00
jimt 92961717d5 Merge pull request #18 from jimt/patch-1
Fix typo.
2013-07-05 04:37:28 -07:00
Jim Tittsler d29c024e5f Fix typo. 2013-07-05 22:54:41 +12:00
jimt fae240c50c Merge pull request #17 from frequency/master
fix off by one slice index when checking SkipDays of rss feed
2013-06-05 12:04:51 -07:00
Zack Bartel 2b83f94cf5 fix off by one slice index 2013-06-05 10:43:16 -07:00
jimt 3ee2213eae Merge pull request #15 from mattn/try_items
Try items in outside of channel node
2013-05-10 05:49:05 -07:00
mattn 833b57154f Handle content:encoded 2013-05-10 04:55:33 -07:00
mattn ed51885f15 Try items in outside of channel node 2013-05-10 19:44:35 +09:00
jimt 3f18008689 Merge pull request #14 from cgrinds/master
Added testdata and test for CDATA bug
2013-03-28 13:13:05 -07:00
Chris Grindstaff ae2d31d0ff Elements that contain multiple CDATA sections are overwritten. Fix is
submitted to go-pkg-xmlx
2013-03-28 16:07:22 -04:00
jimt 583ce7f9e9 Merge pull request #13 from cgrinds/master
Added testdata, rss author parsing and a slight refactor to make reading from files possible
2013-03-28 02:45:17 -07:00
Chris Grindstaff 9a77aeb2f6 Add testdata
RSS author parsing
2013-03-27 23:05:27 -04:00
jimt 87efb1906c Merge pull request #12 from cgrinds/patch-1
Read author for atom item
2013-03-27 13:00:53 -07:00
Chris Grindstaff c8e23415d1 Read author for atom item 2013-03-27 16:05:18 -03:00
jimt 07f688de02 Merge pull request #11 from anschelsc/master
Use the new xmlx changes
2013-03-19 12:56:04 -07:00
Anschel Schaffer-Cohen cccaf19d93 Use upstream xmlx now that it has merged. 2013-03-19 14:12:11 -04:00
Anschel Schaffer-Cohen 069bcf4dc8 Use more flexible go-pkg-xmlx.
If/when the changes are pulled into jteeuwen's branch I can change the
path back.
2013-03-19 10:03:21 -04:00
jim teeuwen 6f6d1c9b3a Fix Feed.SecondsTilUpdate(). Last patch failed to convert stored time to seconds properly. Fixes issue #10. 2012-03-13 16:46:42 +01:00
jim teeuwen 057769eeb5 Rename "examples" folder to "testdata". This causes "go install" to ignore it when building the lib. Example binaries do not get installed in $GOBIN this way. They do not belong there. 2012-03-04 12:52:37 +01:00
jim teeuwen 3575e810e5 Rename "examples" folder to "testdata". This causes "go install" to ignore it when building the lib. Example binaries do not get installed in $GOBIN this way. They do not belong there. 2012-03-04 12:52:29 +01:00