Needs review
Project:
Chaos Tool Suite (ctools)
Version:
7.x-1.x-dev
Component:
Plugins system
Priority:
Normal
Category:
Feature request
Assigned:
Unassigned
Reporter:
Created:
12 Apr 2013 at 14:49 UTC
Updated:
2 Jan 2015 at 13:54 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
zestagio commentedComment #3
zestagio commentedComment #5
zestagio commentedComment #6
andypostAwesome idea! I think this should be implemented as 1 plugin to allow choose a response type a-la page_manager has
And use this setting in summary and access check
You need to implement settings_form()
Comment #7
zestagio commentedandypost, thanks for review. In the near future I will put a new version
Sorry for my English.
Comment #8
zestagio commentedComment #9
andypostJust small nitpicks
403 => t(),
404 => t(),
is enough here, no reason to use string for key
return $conf['code'] == drupal_get_http_header('status');Seems cleaner
Comment #10
zestagio commentedI used string for keys, because keys are http headers for easily compare. In the new patch, I added a description of why to use a string keys.
Sorry for my English.
Comment #11
zestagio commentedComment #12
andypostAwesome!
Comment #13
japerryHmm needs more review... If I remember correctly, I think D8 has a plugin like this, we'd probably want to backport.