Commit message (Collapse) | Author | Age | Files | Lines | |
---|---|---|---|---|---|
* | Include AV::Layouts directly in AM::Base | Łukasz Strzałkowski | 2013-12-05 | 1 | -0/+2 |
| | | | | No need to do this in railtie as AM depends on AV either way | ||||
* | `mail()` without arguments is a getter for the current mail. | Yves Senn | 2013-12-02 | 1 | -0/+2 |
| | | | | | | | | | | | | | | This behavior is documented in our guides (http://edgeguides.rubyonrails.org/action_mailer_basics.html#action-mailer-callbacks) but was broken in the past. This commit short curcuits the `mail` method if: 1. mail() was previously called 2. no headers are passed 3. no block is passed Closes #13090. /cc @pixeltrix | ||||
* | Ruby's new Hash syntax applied in actionmailer | Mr A | 2013-11-14 | 1 | -2/+2 |
| | |||||
* | calculate the ivars to remove in advance as a set and cache them in a | Aaron Patterson | 2013-11-06 | 1 | -1/+5 |
| | | | | | | | constant. `view_assigns` can use the precalculated sets and remove instance variables without allocating any extra arrays | ||||
* | Merge pull request #12591 from vipulnsward/remove_mail_merge | Yves Senn | 2013-10-20 | 1 | -3/+3 |
|\ | | | | | Remove extra variable creation and merge. | ||||
| * | Remove extra variable creation and merge. | Vipul A M | 2013-10-20 | 1 | -3/+3 |
| | | |||||
* | | Instrument the generation of Action Mailer messages | Daniel Schierbeck | 2013-10-20 | 1 | -4/+11 |
|/ | | | | | | The processing of outbound mail is instrumented with the key `process.action_mailer`. The payload includes the mailer name as well as the mailer method. | ||||
* | Revert "Port all remaining self.protected_instance_variables to class methods" | Łukasz Strzałkowski | 2013-09-02 | 1 | -4/+2 |
| | | | | This reverts commit 7de994fa215e9f4c2856d85034bc4dd7b65d0c01. | ||||
* | Port all remaining self.protected_instance_variables to class methods | Łukasz Strzałkowski | 2013-08-29 | 1 | -2/+4 |
| | |||||
* | Remove hard require to ActionView from ActionMailer | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 |
| | |||||
* | Load AV::Layout to AM::Base in railties | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+0 |
| | |||||
* | Include AbsC::Rendering interface in ActionMailer | Łukasz Strzałkowski | 2013-08-25 | 1 | -0/+3 |
| | |||||
* | Do not include AV::Rendering | Łukasz Strzałkowski | 2013-08-25 | 1 | -1/+0 |
| | | | | It's already included in Layouts, makes no sense to doubly include it | ||||
* | Integrate ActionMailer with ActionView | Łukasz Strzałkowski | 2013-08-25 | 1 | -2/+5 |
| | |||||
* | don't convert mailer default values to procs | Alex Tsukernik | 2013-07-22 | 1 | -1/+1 |
| | | | | | | Invoke mailer defaults as procs only if they are procs, do not convert with to_proc. That an object is convertible to a proc does not mean it's meant to be always used as a proc. Fixes #11533 | ||||
* | Revert "nodoc ActionMailer module from lib/action_mailer/*.rb" | Vijay Dev | 2013-05-19 | 1 | -1/+1 |
| | | | | | | | | This reverts commit 328e876a1569058e47eaf7ea3f4f045a83b585fa. Reason: Let's not nodoc without reason. [ci skip] | ||||
* | Fix documentation for ActionMailer template names. | Megan Bowra-Dean | 2013-05-17 | 1 | -4/+4 |
| | | | | ActionMailer templates that are not plain text do not require *.text.* in the name and will fail to be picked up automatically if it is included in the name. | ||||
* | nodoc ActionMailer module from lib/action_mailer/*.rb | Zachary Scott | 2013-05-15 | 1 | -1/+1 |
| | |||||
* | Document ActionMailer::Base#set_content_type | Zachary Scott | 2013-05-12 | 1 | -0/+9 |
| | |||||
* | RDoc style for ActionMailer::Base#mail | Zachary Scott | 2013-05-10 | 1 | -23/+25 |
| | |||||
* | Document ActionMailer::Base::default, ::mailer_name, and #mailer_name | Zachary Scott | 2013-05-10 | 1 | -14/+25 |
| | |||||
* | Fix grammar on AM.mailer_name documentation. | Hendy Tanata | 2013-05-06 | 1 | -1/+1 |
| | |||||
* | Add some documentation for AM.mailer_name | Prem Sichanugrist | 2013-05-03 | 1 | -0/+2 |
| | |||||
* | Remove superfluous `self.` in Action Mailer mail method documetation | Alexey Vakhov | 2013-04-23 | 1 | -3/+3 |
| | | | | | | Notation with `self` was sugestted at 0b05acd, when self.defaults was extlib_inheritable_accessor. But now all Action Mailer examples are without this self. It's nicer. | ||||
* | Cosmetic fixes in AM::Base docs. Missed spaces in OpenSSL constants ↵ | Alexey Vakhov | 2013-04-23 | 1 | -2/+2 |
| | | | | | | enumerations. Injected at a1639ad. | ||||
* | Send paths argument as an array rather than converting it internally | Carlos Antonio da Silva | 2013-01-28 | 1 | -3/+3 |
| | |||||
* | Allow passing interpolations to `#default_i18n_subject`, e.g.: | Olek Janiszewski | 2013-01-24 | 1 | -2/+3 |
| | | | | | | | | | | | | | | | # config/locales/en.yml en: user_mailer: welcome: subject: 'Hello, %{username}' # app/mailers/user_mailer.rb class UserMailer < ActionMailer::Base def welcome(user) mail(subject: default_i18n_subject(username: user.name)) end end | ||||
* | Fix missing spaces before and after equal signs in method definitions | Robin Dupret | 2013-01-03 | 1 | -2/+2 |
| | |||||
* | Move background jobs to the 'jobs' branch until fully baked. Not shipping ↵ | Jeremy Kemper | 2012-12-21 | 1 | -9/+2 |
| | | | | with Rails 4.0. | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-12-15 | 1 | -1/+1 |
|\ | | | | | | | | | | | Conflicts: actionpack/lib/action_view/helpers/form_helper.rb railties/lib/rails/info_controller.rb | ||||
| * | use default_options= instead of default to avoid confusion | Francesco Rodriguez | 2012-12-09 | 1 | -1/+1 |
| | | |||||
* | | Remove not used variable warnings from AM and AC | Carlos Antonio da Silva | 2012-12-11 | 1 | -2/+1 |
| | | |||||
* | | Merge pull request #8450 from senny/8448_mailer_return_values | Rafael Mendonça França | 2012-12-10 | 1 | -1/+4 |
|\ \ | |/ |/| | the return value of mailer methods should not be relevant | ||||
| * | The return value from mailer methods is not relevant. | Yves Senn | 2012-12-10 | 1 | -1/+4 |
| | | |||||
* | | use _action callbacks in actionmailer | Francesco Rodriguez | 2012-12-08 | 1 | -4/+4 |
| | | |||||
* | | Cleans and removes useless 'Examples' tag [ci skip] | Alvaro Pereyra | 2012-12-01 | 1 | -2/+0 |
|/ | |||||
* | Tiny refactor | Santiago Pastorino | 2012-11-29 | 1 | -4/+1 |
| | |||||
* | Explicit multipart messages respect :parts_order | Nate Berkopec | 2012-11-19 | 1 | -8/+6 |
| | | | | | | | | | As issue #7978, the order in which ActionMailer sends multipart messages could be unintentionally overwritten if a block is passed to the mail method. This changes the mail method such that :parts_order is always respected, regardless of whether a block is passed to mail. | ||||
* | Actionmailer to 1.9 hash syntax | AvnerCohen | 2012-11-08 | 1 | -9/+9 |
| | |||||
* | Revert "Support `Mailer.deliver_foo(*args)` as a synonym for ↵ | Jon Leighton | 2012-10-30 | 1 | -3/+0 |
| | | | | | | | | | | | | `Mailer.foo(*args).deliver`." This reverts commit 7e0cf563639bc7508da381b1b8321c7a89be1aa8. Conflicts: actionmailer/CHANGELOG.md See discussion at https://github.com/rails/rails/commit/7e0cf563639bc7508da381b1b8321c7a89be1aa8#commitcomment-2075489 | ||||
* | Do not render views when mail() isn't called. (NullMail refactoring) | Yves Senn | 2012-10-28 | 1 | -1/+13 |
| | |||||
* | Update actionmailer with new hash syntax. | Kirill Nikitin | 2012-10-07 | 1 | -42/+42 |
| | |||||
* | Support `Mailer.deliver_foo(*args)` as a synonym for ↵ | Jon Leighton | 2012-09-28 | 1 | -0/+3 |
| | | | | | | | `Mailer.foo(*args).deliver`. This makes it easy to write e.g. `Mailer.expects(:deliver_foo)` when testing code that calls the mailer. | ||||
* | Revert "update ActionMailer::Base documentation [ci skip]" | Rafael Mendonça França | 2012-09-21 | 1 | -258/+216 |
| | | | | | | | | | | This reverts commit f5654e78b248ecc90e8556366c927e1176c2428f. Conflicts: actionmailer/lib/action_mailer/base.rb Reason: @frodsan asked me to revert since this change is breaking the documentation in the edge API site | ||||
* | Merge branch 'master' of github.com:lifo/docrails | Vijay Dev | 2012-09-21 | 1 | -220/+260 |
|\ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: actionmailer/lib/action_mailer/base.rb activesupport/lib/active_support/configurable.rb activesupport/lib/active_support/core_ext/module/deprecation.rb guides/source/action_controller_overview.md guides/source/active_support_core_extensions.md guides/source/ajax_on_rails.textile guides/source/association_basics.textile guides/source/upgrading_ruby_on_rails.md While resolving conflicts, I have chosen to ignore changes done in docrails at some places - these will be most likely 1.9 hash syntax changes. | ||||
| * | remove nodoc [ci skip] | Vijay Dev | 2012-09-21 | 1 | -1/+1 |
| | | |||||
| * | fix Hash#slice code example [ci skip] | Francesco Rodriguez | 2012-09-18 | 1 | -1/+1 |
| | | |||||
| * | update ActionMailer::Base documentation [ci skip] | Francesco Rodriguez | 2012-09-17 | 1 | -266/+323 |
| | | |||||
* | | spurious space I saw in passing | Xavier Noria | 2012-09-18 | 1 | -1/+1 |
| | | |||||
* | | Use synchronous queue by default. Separate queued message delivery jobs from ↵ | Jeremy Kemper | 2012-09-16 | 1 | -0/+2 |
| | | | | | | | | the queued message wrappers so the queue itself needn't be marshaled (due to queue reference QueuedMessage). |