Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | [docs] fix ActionDispatch documentation | Hrvoje Šimić | 2017-03-13 | 1 | -1/+1 |
| | |||||
* | Privatize unneededly protected methods in Action Pack | Akira Matsuda | 2016-12-24 | 1 | -1/+2 |
| | |||||
* | Add missing `+` around a some literals. | bogdanvlviv | 2016-10-27 | 1 | -1/+1 |
| | | | | | | Mainly around `nil` [ci skip] | ||||
* | normalizes indentation and whitespace across the project | Xavier Noria | 2016-08-06 | 1 | -7/+7 |
| | |||||
* | applies new string literal convention in actionpack/lib | Xavier Noria | 2016-08-06 | 1 | -8/+8 |
| | | | | | The current code base is not uniform. After some discussion, we have chosen to go with double quotes by default. | ||||
* | Make flash messages cookie compatible with Rails 4 | Rafael Mendonça França | 2016-05-06 | 1 | -1/+1 |
| | | | | | | | | In #18721 we removed the discard key from the session hash used to flash messages and that broke compatibility with Rails 4 applications because they try to map in the discarded flash messages and it returns nil. Fixes #24726. | ||||
* | Fix request.reset_session for API controllers | Jon Moss | 2016-03-19 | 1 | -0/+5 |
| | | | | | | | | | | | | Due to that `ActionDispatch::Flash` (the flash API's middleware) is not included for API controllers, the `request.reset_session` method, which relies on there being a `flash=` method which is in fact defined by the middleware, was previously breaking. Similarly to how add46482a540b33184f3011c5c307f4b8e90c9cc created a method to be overridden by the flash middleware in order to ensure non-breakage, this is how flashes are now reset. Fixes #24222 | ||||
* | pull the flash methods in to their own module | Aaron Patterson | 2015-09-25 | 1 | -34/+38 |
| | | | | | | We only want to activate flash when the user has enabled it. Api servers don't use flash, so add an empty implementation to the base Request object. | ||||
* | commit the flash after the controller finishes being serviced | Aaron Patterson | 2015-09-25 | 1 | -10/+16 |
| | | | | | Committing the flash needs to happen in order for the session to be written correctly, so lets guarantee that it actually does happen. | ||||
* | move flash committing to the request object. | Aaron Patterson | 2015-09-25 | 1 | -12/+1 |
| | | | | | I'm doing this so that we can commit the flash to the session object Out of Band of the flash middleware | ||||
* | ask the request object for the session | Aaron Patterson | 2015-09-22 | 1 | -1/+1 |
| | | | | | The flash middleware shouldn't know how to look up the session object. Just ask the request for that information. | ||||
* | stop inheriting from Rack::Request | Aaron Patterson | 2015-09-04 | 1 | -1/+1 |
| | | | | | | Just include the modules necessary in the Request object to implement the things we need. This should make it easier to build delegate request objects because the API is smaller | ||||
* | remove usage of `@env` | Aaron Patterson | 2015-08-23 | 1 | -3/+5 |
| | | | | | try to remove dependencies on `@env` so we can have more flexible internals | ||||
* | use a request object in the session middleware | Aaron Patterson | 2015-08-22 | 1 | -1/+1 |
| | | | | | This commit allows us to use one request object rather than allocating multiple request objects to deal with the session. | ||||
* | move flash hash access to methods on the request object | Aaron Patterson | 2015-08-07 | 1 | -2/+11 |
| | |||||
* | Migrate old flash behaviour | Samuel Cochran | 2015-01-30 | 1 | -0/+6 |
| | |||||
* | Discard from flash before persisting in session | Samuel Cochran | 2015-01-29 | 1 | -17/+17 |
| | |||||
* | adding nodoc to private methods | Jonathan Cutrell | 2014-11-23 | 1 | -2/+2 |
| | |||||
* | Adding simple docs for ActionDispatch::Flash::FlashHash#to_session_value | Jonathan Cutrell | 2014-11-23 | 1 | -1/+4 |
| | |||||
* | Make FlashHash#key? work with symbol and string | Rafael Mendonça França | 2014-11-11 | 1 | -1/+1 |
| | | | | Closes #17586 | ||||
* | flash doesn't pass objects #15522 [ci skip] | Nishant Modak | 2014-07-01 | 1 | -3/+6 |
| | |||||
* | Missed FlashHash#replace | Godfrey Chan | 2014-02-11 | 1 | -1/+1 |
| | |||||
* | Fixed broken flash tests | Godfrey Chan | 2014-02-11 | 1 | -1/+1 |
| | |||||
* | Stringify the incoming hash in FlashHash | Guillermo Iguaran | 2014-02-11 | 1 | -1/+3 |
| | | | | | Stringify the incoming as well to handle incoming symbol keys from marshalled sessions | ||||
* | Convert FlashHash in a Hash with indifferent access | Guillermo Iguaran | 2014-02-11 | 1 | -4/+15 |
| | |||||
* | Fixing build broken by this change | Arun Agrawal | 2013-05-13 | 1 | -2/+2 |
| | | | | c43ca06ca091fc09e2c86bb051ac92b648f12b64 | ||||
* | Code cleanup for ActionDispatch::Flash#call | Julian Vargas | 2013-05-12 | 1 | -9/+3 |
| | | | | | | | The nested `if` was replaced by using `presence` which takes account for the given hash when it is `nil` or when it is empty. The `else` was removed because what it was doing was to assign to `env[KEY]` the value it already had. | ||||
* | extract alert= and notice= examples to FlashHash#now [ci skip] | Francesco Rodriguez | 2013-01-03 | 1 | -17/+15 |
| | |||||
* | Add examples `alert=` and `notice=`, using memes | lambda_ | 2013-01-03 | 1 | -0/+10 |
| | |||||
* | Remove nodoc from FlashHash #[]= [ci skip] | Pablo Torres | 2012-12-01 | 1 | -1/+1 |
| | |||||
* | Revert "Remove trailing whitespaces" | Florent Guilleux | 2012-12-01 | 1 | -1/+1 |
| | | | | This reverts commit 90c887fa7d0c454b7533e208daefc342dea4d5f3. | ||||
* | Remove trailing whitespaces | Florent Guilleux | 2012-12-01 | 1 | -1/+1 |
| | |||||
* | Store FlashHashes in the session as plain hashes rather than custom objects ↵ | James Coglan | 2012-11-09 | 1 | -10/+24 |
| | | | | | | | | | | | | | | | | | | | | with unstable class names and instance variables. Refactor FlashHash to take values for its ivars in the constructor, to pretty up FlashHash.from_session_value. Remove stale comment on FlashHash: it is no longer Marshaled in the session so we can change its implementation. Remove blank lines I introduced in controller/test_case.rb. Unit tests for FlashHash#to_session_value. Put in a compatibility layer to accept FlashHash serializations from Rails 3.0+. Test that Rails 3.2 session flashes are correctly converted to the new format. Remove code path for processing Rails 3.0 FlashHashes since they can no longer deserialize. Fix session['flash'] deletion condition: it will never be empty?, it will either be nil or a hash with 'discard' and 'flashes' keys. | ||||
* | Removing ==Examples and last blank lines of docs from actionpack | Francesco Rodriguez | 2012-05-15 | 1 | -1/+1 |
| | |||||
* | make sure the superclass matches so load order does not matter | Aaron Patterson | 2012-05-04 | 1 | -1/+1 |
| | |||||
* | bread AD::Request::Session to it's own file, consolidate HASH OF DOOM lookups | Aaron Patterson | 2012-05-03 | 1 | -1/+1 |
| | |||||
* | Merge pull request #6083 from brainopia/remove_unused_closed_ivars | José Valim | 2012-04-30 | 1 | -1/+0 |
|\ | | | | | Remove a couple of unused ivars left from previous refactoring | ||||
| * | Remove unused ivars left from close checks | brainopia | 2012-04-30 | 1 | -1/+0 |
| | | | | | | | | These ivars were missed in d142572567 when close checks were removed | ||||
* | | fix the Flash middleware loading the session on every request (very ↵ | Will Bryant | 2012-04-28 | 1 | -6/+3 |
|/ | | | | dangerous especially with Rack::Cache), it should only be loaded when the flash method is called | ||||
* | Fix example in flash middleware | Alexey Vakhov | 2012-04-17 | 1 | -1/+1 |
| | |||||
* | simplify sweep now that discard and flashes are in sync | Aaron Patterson | 2011-12-28 | 1 | -8/+2 |
| | |||||
* | mutations on the underlying hash should also mutate the discard set | Aaron Patterson | 2011-12-28 | 1 | -3/+5 |
| | |||||
* | rename @used to something a bit more meaningful | Aaron Patterson | 2011-12-28 | 1 | -7/+7 |
| | |||||
* | mutations can't be done without the consent of our proxy object. This | Aaron Patterson | 2011-12-28 | 1 | -3/+0 |
| | | | | is one benefit of choosing composition over inheritance. | ||||
* | again, use Set#subtract | Aaron Patterson | 2011-12-28 | 1 | -1/+1 |
| | |||||
* | Use Set#subtract and Set#merge for keeping track of used / unused keys | Aaron Patterson | 2011-12-28 | 1 | -15/+4 |
| | |||||
* | fix method visibility | Aaron Patterson | 2011-12-28 | 1 | -15/+15 |
| | |||||
* | no need for bang bang :bomb: | Aaron Patterson | 2011-12-28 | 1 | -1/+1 |
| | |||||
* | Get rid of the close checks since we cannot reliably close the session anyway. | José Valim | 2011-12-16 | 1 | -6/+0 |
| | |||||
* | Add some implementation docs. closes #3298. closes #2509. | José Valim | 2011-10-12 | 1 | -0/+4 |
| |