Support from Acquia helps fund testing for Drupal Acquia logo

Comments

DuaelFr’s picture

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

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

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

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’s picture

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

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.