Unused var removed #4614

Merged
merged 1 commit into from Jan 23, 2012

Conversation

Projects
None yet
4 participants
@arunagw
Member

arunagw commented Jan 23, 2012

No description provided.

josevalim added a commit that referenced this pull request Jan 23, 2012

@josevalim josevalim merged commit 6785001 into rails:master Jan 23, 2012

@@ -19,7 +19,7 @@ def reset_counters(id, *counters)
counters.each do |association|
has_many_association = reflect_on_association(association.to_sym)
- expected_name = if has_many_association.options[:as]

This comment has been minimized.

Show comment Hide comment
@vijaydev

vijaydev Jan 23, 2012

Member

I think the whole if..else can go.

@vijaydev

vijaydev Jan 23, 2012

Member

I think the whole if..else can go.

This comment has been minimized.

Show comment Hide comment
@carlosantoniodasilva

carlosantoniodasilva Jan 23, 2012

Owner

Yeah, seems like it's not being used at all.

@carlosantoniodasilva

carlosantoniodasilva Jan 23, 2012

Owner

Yeah, seems like it's not being used at all.

This comment has been minimized.

Show comment Hide comment
@carlosantoniodasilva

carlosantoniodasilva Jan 23, 2012

Owner

I've sent a pull request removing this part of code.

@carlosantoniodasilva

carlosantoniodasilva Jan 23, 2012

Owner

I've sent a pull request removing this part of code.

carlosantoniodasilva added a commit to carlosantoniodasilva/rails that referenced this pull request Jan 23, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment