aboutsummaryrefslogtreecommitdiffstats
path: root/activesupport/lib/active_support/callbacks.rb
Commit message (Collapse)AuthorAgeFilesLines
* sync __run_callbacks with ruby-trunkSokolov Yura2012-02-201-1/+1
| | | | | | https://bugs.ruby-lang.org/projects/ruby-trunk/repository/revisions/34580 In this revision behavior of respond_to? had changed: now to ask about protected method one should pass second argument `true`
* Fix GH #4344. A defined callback in extended module is called too.kennyj2012-01-241-1/+1
| | | | | | Conflicts: activesupport/test/callbacks_test.rb
* Revert "AS::Callbacks: remove __define_runner"José Valim2012-01-041-3/+16
| | | | | | Runners are used internally. This reverts commit 40c8aa706adbb60ff9d289016fb9020d48969fea.
* AS::Callbacks: remove __define_runnerBogdan Gusiev2012-01-031-16/+3
|
* Refactor AS::CallbacksBogdan Gusiev2012-01-031-6/+8
| | | | | Extracted `__reset_runner` from `__define_runner` And call it in proper places
* AS::Callbacks: improved __define_runner performanceBogdan Gusiev2012-01-031-3/+2
|
* AS::Callbacks: improve __define_runner perfomanceBogdan Gusiev2012-01-031-10/+11
|
* Log 'Filter chain halted as CALLBACKNAME rendered or redirected' every time ↵José Valim2011-11-301-4/+14
| | | | a before callback halts.
* AS::Callbacks::Callback#_compile_option refactoredBogdan Gusiev2011-11-111-14/+12
|
* AS::Callbacks.__run_keyed_callback: remove unused cacheBogdan Gusiev2011-11-101-10/+6
|
* self.class.name -> self.name (we are already inside a class).José Valim2011-11-101-1/+1
|
* AS::Callbacks#_define_runner refactoredBogdan Gusiev2011-11-101-23/+22
| | | | | Incapsulate the logic of keyed callback method definition and execution in the separated method.
* Remove `#among?` from Active SupportPrem Sichanugrist2011-04-131-1/+1
| | | | | | After a long list of discussion about the performance problem from using varargs and the reason that we can't find a great pair for it, it would be best to remove support for it for now. It will come back if we can find a good pair for it. For now, Bon Voyage, `#among?`.
* Change Object#either? to Object#among? -- thanks to @jamesarosen for the ↵David Heinemeier Hansson2011-04-121-1/+1
| | | | suggestion!
* Using Object#in? and Object#either? in various placesPrem Sichanugrist2011-04-111-1/+2
| | | | There're a lot of places in Rails source code which make a lot of sense to switching to Object#in? or Object#either? instead of using [].include?.
* Prevent callbacks in child classes from being executed more than once.Rolf Timmermans2011-03-131-1/+1
|
* adds a couple of missing AS dependenciesRolf Timmermans2011-03-121-0/+1
|
* Allow access to a callback event's return result from around callbacksAaron Pfeifer2011-03-081-2/+8
|
* Merge branch 'master' of git://github.com/lifo/docrailsXavier Noria2011-01-201-113/+116
|\
| * editing comments regarding rescuable optoin in AS callbacksNeeraj Singh2011-01-121-2/+3
| |
| * Expand and clarify AS::Callbacks docs.John Firebaugh2011-01-091-113/+115
| |
* | removing usesless variable assignmentsAaron Patterson2011-01-181-1/+5
|/
* expand on set_callback method to explain that inNeeraj Singh2010-12-211-0/+4
| | | | some cases an implicit :before is assumed
* Replace nested ifs with case/whenPavel Gorbokon2010-12-071-46/+44
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* Ommit nil in method callPavel Gorbokon2010-12-071-1/+1
| | | | Signed-off-by: Santiago Pastorino <santiago@wyeworks.com>
* bye bye extlib_inheritable_*, AS callbacks now using class_attributeJosh Kalderimis2010-11-181-9/+13
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Correct documentation that references how *_filter of action_pack are ↵Marcelo Giorgi2010-10-031-2/+2
| | | | translated into set_callback/run_callback's (from active_support) invocations.
* after_create in ActiveModel should in the order specifiedNeeraj Singh2010-09-271-1/+1
| | | | | | [#5650 state:resolved] Signed-off-by: José Valim <jose.valim@gmail.com>
* lifecycle should be two words, life cycleJaime Iniesta2010-08-261-1/+1
|
* Deletes trailing whitespaces (over text files only find * -type f -exec sed ↵Santiago Pastorino2010-08-141-1/+1
| | | | 's/[ \t]*$//' -i {} \;)
* adding an example of skipping a callbackNeeraj Singh2010-08-061-1/+5
|
* Merge remote branch 'docrails/master'Xavier Noria2010-08-051-1/+2
|\
| * documents that :terminator is a string to be eval'ed, and that it sees the ↵Xavier Noria2010-08-051-1/+2
| | | | | | | | result variable
| * Revert "correcting the documentation that default to false. it does not ↵Xavier Noria2010-08-051-4/+5
| | | | | | | | | | | | | | | | default to false. returing either false or nil will not halt the chain unless :terminator is explicitly mentioned" This reverts commit cd87cf771acb90c43cd56e5f038fc345a69790f9. Reason: it does default to "false".
| * correcting the documentation that default to false. it does not default to ↵Neeraj Singh2010-08-051-5/+4
| | | | | | | | false. returing either false or nil will not halt the chain unless :terminator is explicitly mentioned
* | Merge remote branch 'docrails/master'Xavier Noria2010-08-051-3/+3
|\|
| * Fix ActiveSupport::Callbacks' define_callbacks and ActiveSupport::Concern ↵Tom Stuart2010-08-051-3/+3
| | | | | | | | documentation to look like native English
* | Merge remote branch 'docrails/master'Xavier Noria2010-08-051-1/+3
|\|
| * adding documentation to ActiveSupport::Concern ht:strictly typed for an ↵Neeraj Singh2010-08-041-1/+3
| | | | | | | | | | | | awesome example some minor documentation changes
* | Reload action_methods in AbstractController after defining new method.Piotr Sarnacki2010-08-041-1/+4
|/ | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* Define a convention for descendants and subclasses.José Valim2010-07-051-2/+2
| | | | | | The former should be symmetric with ancestors and include all children. However, it should not include self since ancestors + descendants should not have duplicated. The latter is symmetric to superclass in the sense it only includes direct children. By adopting a convention, we expect to have less conflict with other frameworks, as Datamapper. For this moment, to ensure ActiveModel::Validations can be used with Datamapper, we should always call ActiveSupport::DescendantsTracker.descendants(self) internally instead of self.descendants avoiding conflicts.
* Change callbacks to automatically include DescendantsTracker and rename ↵José Valim2010-06-191-3/+7
| | | | descendents to descendants.
* edit pass: the names of Rails components have a space, ie, "Active Record", ↵Xavier Noria2010-06-141-1/+1
| | | | not "ActiveRecord"
* edit pass in #define_callbacks rdocXavier Noria2010-06-141-20/+22
|
* adding to the :kind documentation for ActiveSupport callbacksNeeraj Singh2010-06-141-0/+5
|
* clearer documentation of how scope applies to ActiveSupport callbacksNeeraj Singh2010-06-141-20/+32
|
* Simplify and optimize callbacks superclass sync.José Valim2010-06-121-54/+21
|
* Eliminate the need to check for superclass changes to the callback stack ↵wycats2010-06-041-3/+6
| | | | each time through the callbacks
* refactor evals and adds some __FILE__ and __LINE__Santiago Pastorino2010-05-201-27/+24
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>
* fix stack trace lines on class_evalSantiago Pastorino2010-04-091-1/+1
| | | | Signed-off-by: José Valim <jose.valim@gmail.com>