I Think it doesn't work with anonymous users... Can someone confirm it ??

CommentFileSizeAuthor
#5 doesn_t_work_without-2383431-5.patch681 bytesjoshtaylor
#4 doesn_t_work_without-2383431-4.patch1.45 KBjoshtaylor

Comments

fra_ore_90’s picture

fra_ore_90 commented

Noone have idea???

joshtaylor’s picture

joshtaylor commented

Confirmed this doesn't work with anonymous users.

The limit is across ALL anonymous users, so anonymous users can only flag 3 items in total.

markwk’s picture

markwk commented
Category: Bug report » Feature request

Wasn't built to handle anonymous users, just logged-in users. Happy to see this feature added.

Changing status to feature request

joshtaylor’s picture

joshtaylor commented
StatusFileSize
new doesn_t_work_without-2383431-4.patch1.45 KB

Here is an initial patch for this.

joshtaylor’s picture

joshtaylor commented
StatusFileSize
new doesn_t_work_without-2383431-5.patch681 bytes

Woops, let's try without the extra cruff.

fra_ore_90’s picture

fra_ore_90 commented

Thanks Josh! This works great!

joshtaylor’s picture

joshtaylor commented
Status: Active » Needs review
joshtaylor’s picture

joshtaylor commented

Hi markwk,

Can I get a review for this?

markwk’s picture

markwk commented

Took a look at code and seems fine conceptually but not be good to get another reviewer to test before committing and breaking other people's sites. Anyone else checked on this?

matysek145’s picture

matysek145 commented

Works well :)

rudins’s picture

rudins commented
Status: Needs review » Reviewed & tested by the community

Confirm, can be committed.

andrewsizz’s picture

andrewsizz commented

also works for me

steveoriol’s picture

steveoriol
Location Grenoble 🇫🇷
commented

Works well for me too, thank you.