public
Description: Support for multiple SMTP servers in ActionMailer
Clone URL: git://github.com/nakajima/many-mailers.git
use Array#shift instead of reversing then popping
nakajima (author)
Wed Jul 23 11:10:05 -0700 2008
commit  bd85ec2cc88a637a69ef2cf5b6e8fda774405454
tree    be005e3cff0eae7b745ba8eda9d6b51170d2f1ef
parent  273c4bc110d376f88c65102b6b5e33b92d9c4a0a
...
13
14
15
16
 
17
18
19
20
21
 
22
23
24
...
13
14
15
 
16
17
18
19
20
 
21
22
23
24
0
@@ -13,12 +13,12 @@ module Animoto
0
       def with_settings(name, options={}, &block)
0
         self.class_inheritable_accessor :smtp_settings
0
         self.smtp_settings = mail_servers[name]
0
- rescue_servers = [options[:retry]].flatten.reverse
0
+ rescue_servers = [options[:retry]].flatten
0
         begin
0
           yield self
0
         rescue => e
0
           raise e if rescue_servers.empty?
0
- retry_server = rescue_servers.pop
0
+ retry_server = rescue_servers.shift
0
           with_settings(retry_server, :retry => rescue_servers, &block)
0
         end
0
         self.smtp_settings = mail_servers[:default]

Comments

    No one has commented yet.