#4877 [PATCH] Refactor requires - Ruby on Rails

Type	To find
responsible:me	tickets assigned to you
tagged:"@high"	tickets tagged @high
milestone:next	tickets in the upcoming milestone
state:invalid	tickets with the state invalid
created:"last week"	tickets created last week
sort:number, importance, updated	tickets sorted by #, importance or updated
Combine keywords for powerful searching.
Use advanced searching »

This project is archived and is in readonly mode.

#4877 ✓invalid

[PATCH] Refactor requires

Reported by Sebastian Martinez | June 16th, 2010 @ 10:04 PM

Found that all files on the folder were being required separately.
I refactored that require.

You flagged this item as spam.
Sebastian Martinez June 16th, 2010 @ 10:32 PM
- Tag changed from “rails3” to “activesupport, rails3”
- Title changed from “Refactor requires” to “[PATCH] Refactor requires”
You flagged this item as spam.
Sebastian Martinez June 20th, 2010 @ 11:13 PM
- Assigned user changed from “Yehuda Katz (wycats)” to “José Valim”
José Valim June 20th, 2010 @ 11:40 PM
Why the way it's currently in Rails is an issue?
You flagged this item as spam.
Sebastian Martinez June 20th, 2010 @ 11:46 PM
It's not wrong, just found that it was requiring all the files on the folder one by one and refactored it.
If we add another file to the integer folder no new require line would be needed this way.
José Valim June 20th, 2010 @ 11:50 PM
- State changed from “new” to “invalid”
I find the current way much more readable. It could make sense though if we had several files.

Help contribute to this project by taking a few moments to create your personal profile. Create your profile »

<h2 style="font-size: 14px">Tickets have moved to Github</h2>

The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>