python-api-client add iter in FeedlyData

should fix #12

Jun 28 '19 22:06 lhoestq

if 'content' in entry: entry, instance of FeedlyData, is not supposed to be an iterable right ? Rather, we should use the contains method that checks for membership in the _json ?

Jun 30 '19 15:06 Edouard360

Well if we only have the contains overloading, then something like for i in entry would do a infinite loop. That's why iter overloading was ok because it also fixes this case

Jun 30 '19 15:06 lhoestq

We're not supposed to do iter this way in the first place, are we ? Wouldn't we rather raise an error if we wanted to take that edge case into account ? Maybe we could put an unit test for your case for i in entry ...

Jun 30 '19 15:06 Edouard360

We're not supposed to do iter this way in the first place, are we ? Wouldn't we rather raise an error if we wanted to take that edge case into account ? Maybe we could put an unit test for your case for i in entry ...

i think this would be a good solution...if people really want some dictionary like behavior on the data, they can call .json and use the underlying data?

Jul 01 '19 17:07 kireet