Stream-Framework icon indicating copy to clipboard operation
Stream-Framework copied to clipboard

feed[:] != feed[:feed.count()] !?

Open nkprince007 opened this issue 6 years ago • 3 comments

The implementation of __getitem__ in BaseFeed class has some bug. It's not returning the last value when [:] is used. And I had to use feed[:feed.count()] to get the entire list. This is very unlike how a python list works.

nkprince007 avatar Dec 05 '18 12:12 nkprince007

@nkprince007 good spot! Feel free to make a PR for this :)

tbarbugli avatar Dec 05 '18 12:12 tbarbugli

Hi, I also experienced this issue while invoking unfollow_feed (https://github.com/tschellenbach/Stream-Framework/blob/e7535857af03c81c760b9265568287816ecac13c/stream_framework/feed_managers/base.py#L222)

@nkprince007 If You haven't created fix for that yet I will try to do this in the evening, if You don't mind

MWMWM avatar Apr 11 '19 11:04 MWMWM

I turns out that in my case the problem was with https://github.com/jamesls/fakeredis - with 'real' redis it works like a charm :)

MWMWM avatar Apr 13 '19 13:04 MWMWM