mahlon/ruby-ezmlm
1
0
Fork 0
Code Issues Pull requests Activity

Don't mutate the caller's argument.

Browse source
This commit is contained in:
Mahlon E. Smith 2017-06-23 10:54:26 -07:00
parent f43372e648
commit 1f2a77373b
1 changed files with 3 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

6
lib/ezmlm/list.rb
View file

@ -70,7 +70,7 @@ class Ezmlm::List
### Returns +true+ if +address+ is a subscriber to this list. ### Returns +true+ if +address+ is a subscriber to this list.
### ###
def include?( addr, section: nil ) def include?( addr, section: nil )
addr.downcase! addr = addr.downcase
file = self.subscription_dir( section ) + Ezmlm::Hash.subscriber( addr ) file = self.subscription_dir( section ) + Ezmlm::Hash.subscriber( addr )
return false unless file.exist? return false unless file.exist?
return file.read.scan( /T([^\0]+)\0/ ).flatten.include?( addr ) return file.read.scan( /T([^\0]+)\0/ ).flatten.include?( addr )
@ -91,7 +91,7 @@ class Ezmlm::List
def subscribe( *addr, section: nil ) def subscribe( *addr, section: nil )
addr.each do |address| addr.each do |address|
next unless address.index( '@' ) next unless address.index( '@' )
address.downcase! address = address.downcase
file = self.subscription_dir( section ) + Ezmlm::Hash.subscriber( address ) file = self.subscription_dir( section ) + Ezmlm::Hash.subscriber( address )
self.with_safety do self.with_safety do
@ -117,7 +117,7 @@ class Ezmlm::List
### ###
def unsubscribe( *addr, section: nil ) def unsubscribe( *addr, section: nil )
addr.each do |address| addr.each do |address|
address.downcase! address = address.downcase
file = self.subscription_dir( section ) + Ezmlm::Hash.subscriber( address ) file = self.subscription_dir( section ) + Ezmlm::Hash.subscriber( address )
self.with_safety do self.with_safety do