Since you don't want your database getting forever more massive for no good reason (it is likely you will only ever be showing the last 5 or 10) it would be good to have an option to limit the maximum number of entries in the recently read table for each user.

For example I can set it to 5 and the database would only ever have 5 entries per user at most.

Comments

zterry95’s picture

zterry95 commented

yes. this is already in consideration.

We will implement this in hook_session_api_cleanup() or hook_cron.

If you are interesting. you can try to patch for this.

more details for hook_session_api_cleanup, check:
http://drupalcode.org/project/session_api.git/blob/refs/heads/7.x-1.x:/s...

pgorecki’s picture

pgorecki commented

This option is already there and should work at least for authenticated users.
Go to admin/config/system/recently-read and fill out: Recently read list length.
It allows to set the maximum number of entires stored for each read content type (per user) in the database.

I'm not sure if it work for anonymous users as well.

rooby’s picture

rooby commented

Sorry, I was only going off my reading of the code.

the variable recently_read_max_entries doesn't seem to be used anywhere in the code.

zterry95’s picture

zterry95 commented

I rewrite the config, add max count limit for each entity.

You will find it in 7.x-3.x-dev.

Your feedback will be highly appreciated.

zterry95’s picture

zterry95 commented
Status: Active » Reviewed & tested by the community

  • zterry95 committed c920ac6 on 8.x-1.x 
    inssue #1973526 by rooby : Add an option to limit the number of recently...
boshtian’s picture

boshtian commented
Issue summary: View changes
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.