CommentFileSizeAuthor
#2 node_limit-configurable_message_and_redirect-1364410-2.patch5.39 KBbrolad

Comments

duaelfr’s picture

duaelfr
he/him
Primary language French
Location Montpellier, France
commented
Status: Active » Needs work

It is a good idea.
If you are able to provide us a patch, feel free to.
I add this on my node_limit's to do list.

brolad’s picture

brolad commented
Issue summary: View changes
StatusFileSize
new node_limit-configurable_message_and_redirect-1364410-2.patch5.39 KB

Added custom message and redirect for each content type and one global message/redirect. Maybe need some improve like separate code for modules... Also I allow for users to have posibility display nothing. I think it can be use for some cases.

brolad’s picture

brolad commented
Status: Needs work » Needs review
Pavan B S’s picture

Pavan B S commented
Assigned: Unassigned » Pavan B S
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community

Thanks for the patch, it is working fine for me

  • adriancid committed 4b1a5f7 on 7.x-1.x authored by Arez 
    Issue #1364410 by Arez, Fidelix, Pavan B S, adriancid: Configurable...

adriancid credited adriancid.

adriancid’s picture

adriancid
Primary language Spanish
Location Qc, Canada 🇨🇦
commented
Status: Reviewed & tested by the community » Fixed

Thanks @all, nice work.

Status: Fixed » Closed (fixed)

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

jordanmagnuson’s picture

jordanmagnuson commented

Custom error message and/or redirect needs to be configurable per limit--not per node type. (Either that, or we need access to more placeholders in the error message: such as the timeframe of the limit, and the actual node limit number for the timeframe).

It's quite possible to have a limit of say 1 forum topic per day, and 10 forum topics per week, and want to show a different message when each of these limits is hit. Currently that's not possible.