Skip to content

Commit

Permalink
Change occurences of 'ASC' to lowercase
Browse files Browse the repository at this point in the history
  • Loading branch information
Blaidd-Drwg committed Jan 16, 2017
1 parent 448737f commit bb0801e
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion app/controllers/events_controller.rb
Expand Up @@ -196,7 +196,7 @@ def upload_material

# GET /event/1/participants_pdf
def participants_pdf
default = {:order_by => 'email', :order_direction => 'ASC'}
default = {:order_by => 'email', :order_direction => 'asc'}
default = default.merge(params)

@application_letters = @event.application_letters_ordered(default[:order_by], default[:order_direction])
Expand Down
2 changes: 1 addition & 1 deletion spec/models/event_spec.rb
Expand Up @@ -186,7 +186,7 @@
@profile2 = FactoryGirl.create(:profile, user: @user2, birth_date: 16.years.ago, first_name:'John')
@application2 = FactoryGirl.create(:application_letter_accepted, user: @user2, event: @event)

expect(@event.application_letters_ordered('first_name','ASC')).to eq([@application1,@application2])
expect(@event.application_letters_ordered('first_name','asc')).to eq([@application1,@application2])
end

it "generates an application letter list ordered by anything else" do
Expand Down

0 comments on commit bb0801e

Please sign in to comment.