| Commit message (Collapse) | Author | Age | Lines |
... | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Problem described in http://seclists.org/fulldisclosure/2013/Sep/145
Pattern taken from https://www.coffeepowered.net/2013/09/26/rails-session-cookies/
|
| | | | | |
|
| |_|/ /
|/| | |
| | | |
| | | | |
It's also used in the web interface and needs to be escaped there.
|
|\ \ \ \ |
|
| | |/ /
| |/| |
| | | |
| | | | |
Better to translate the full string
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Use the same msgids so that translators only have
to translate one thing.
|
|/ / / |
|
|\ \ \ |
|
| | | | |
|
| | | |
| | | |
| | | |
| | | | |
Issue some security-related headers by default.
|
| | |/
| |/|
| | |
| | | |
This prevents browsers from storing and filling passwords.
|
| | | |
|
|\ \ \
| |_|/
|/| | |
|
| | | |
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Only return 3 similar requests to prevent the suggestions swamping the
page
|
| | |
| | |
| | |
| | | |
Use the per_page parameter to limit the results returned
|
| | |
| | |
| | |
| | |
| | | |
Only render the request_search_ahead_results div when a search has been
made.
|
| | | |
|
| | |
| | |
| | |
| | | |
No point trying to render the results if there aren’t any to render
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Each instead of for
Unless instead of if !
.any? instead of size > 0
Line length
|
| | | |
|
| | |
| | |
| | |
| | |
| | | |
Action now supports the `request_from` param as per the Xapian filtering
system to filter search typeaheads by public body
|
| | | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
7a3b462f41321034cbdd2c83707f739a442e83c6.
The string with the unsubstituted variable pattern was being made
available for translation, instead of the translatable version.
|
|\ \ \ |
|
| | | |
| | | |
| | | |
| | | | |
It's required in [PublicBody#type_of_authority]( https://github.com/mysociety/alaveteli/blob/rails-3-develop/app/models/public_body.rb#L343-L347)
|
|\ \ \ \
| |/ / /
|/| | | |
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Now that the form submits to a URL containing the parent object’s id
these are no longer required
See:
- 259a0fd588e198d2a826ef4ac09284c610005387
- d5c7e08d58dc5ad52f16eccd9c456543503e4360
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Easier to check against the assigned variable, but the spec still fails
because an internal error is raised if the rule is not associated with
a User or InfoRequest
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Use resource-oriented form style, building against the censor rule
instance created in the controller
- Use the URL assigned by the controller based on what the censor rule
is being created against
- Use the new route for the buttons directing the admin to the form
- Use consistent styling for button labelling
- Use each instead of for
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
- Add linebreaks to break up sections
- Remove unused parentheses
- Use unless instead of if !
|
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Direct the (re-rendered) form at the correct route for the association
(or use the generic route if the rule is being created for some other
reason)
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Build the CensorRule against the object that is found by an additional
parameter (either :info_request_id or :user_id)
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Direct the form at the correct route for the association (or use the
generic route if the rule is being created for some other reason)
|
| | | |
| | | |
| | | |
| | | |
| | | | |
Build the CensorRule against the object that is found by an additional
parameter (either :info_request_id or :user_id)
|
| | | | |
|
|\ \ \ \
| | | | |
| | | | |
| | | | | |
ssh://git.mysociety.org/data/git/public/alaveteli into rails-3-develop
|
| | | | | |
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Was initially counting the number of headings rather than the number
of categories inside each heading.
|
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Make the forms for editing categories and category headings more uniform
with other admin pages
- Put the dangerous actions in a well
- Remove large well for returning to the index page
|
|\ \ \ \ \
| |/ / / /
|/| | | | |
|
| | | | | |
|
|\ \ \ \ \
| | | | | |
| | | | | |
| | | | | | |
rails-3-develop
|
| | | | | |
| | | | | |
| | | | | |
| | | | | | |
PublicBodyCategories
|
| |/ / / /
| | | | |
| | | | |
| | | | | |
interface
|
| |/ / /
|/| | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
The chevron didn't illustrate that the heading had any category
children. The app uses the badge pattern elsewhere.
Adds minor style improvements:
- Pad erb tags
- Use each instead of for
Fixes https://github.com/mysociety/alaveteli/issues/1962
|
|\ \ \ \ |
|
| |/ / /
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Not clear to the user why the form field is sometimes present and
sometimes not present.
The Category tag field may only be modified if authorities have not yet
been assigned to the Category. This commit explains this and disables
the field if this is true.
|
| | | |
| | | |
| | | | |
Also use double quotes rather than string escaping
|