- 22 8月, 2015 2 次提交
-
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
I'm going to change the constructor, so it's easier to do it in one place.
-
- 21 8月, 2015 1 次提交
-
-
由 Aaron Patterson 提交于
-
- 19 8月, 2015 2 次提交
-
-
由 Aaron Patterson 提交于
nobody should be touching the routes hash without going through the NamedRouteCollection object.
-
由 Aaron Patterson 提交于
-
- 18 8月, 2015 4 次提交
-
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
I want to change the real constructor to take a particular parameter for matching the request method
-
由 Aaron Patterson 提交于
The string we create is almost always the same, so rather than joining all the time, lets join once, then reuse that string everywhere.
-
- 16 8月, 2015 1 次提交
-
-
由 Aaron Patterson 提交于
We should build the routes using the user facing API which is `Mapper`. This frees up the library internals to change as we see fit. IOW we shouldn't be testing internals.
-
- 15 8月, 2015 12 次提交
-
-
由 Aaron Patterson 提交于
The outer router object already keeps a hash of named routes, so we should just use that.
-
由 Aaron Patterson 提交于
refactor the tests with a backwards compatible method call so we can rm add_route2 from the journey router
-
由 Aaron Patterson 提交于
then we can let the mapping object derive stuff that the Route object needs.
-
由 Aaron Patterson 提交于
now that we aren't doing options manipulations, we can just pass the mapping object down and read values from it.
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
now we don't need to add it to a hash and delete it from the hash later just to pass it around
-
由 Aaron Patterson 提交于
`add_route` needs the AST, so rather than shove it in a hash and delete later, lets move parsing up the stack so we can pass down later
-
由 Aaron Patterson 提交于
also change the feeler to subclass AD::Request so that it has all the methods that Request has
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
Eventually I want to eliminate the FakeSet test class
-
由 Aaron Patterson 提交于
I'm going to reimplement this using route objects, so it will be easier if we just change ast access to go through a method rather than hashes
-
由 Aaron Patterson 提交于
we don't need to keep adding it and deleting if from hashes.
-
- 14 8月, 2015 7 次提交
-
-
由 Aaron Patterson 提交于
This was a useless object. We can just directly construct a Path::Pattern object without a Strexp object.
-
由 Aaron Patterson 提交于
the caller already has it, there is no reason to pack it in to an object and just throw that object away.
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
the caller already has access to `as`, so we can stop passing it around.
-
由 Aaron Patterson 提交于
the same value that is extracted from the options hash earlier is returned, so we don't need to pass it in in the first place. The caller already has the data, so stop passing it around.
-
由 Aaron Patterson 提交于
this way we don't have to mutate the options hash so far away from where the user passed it in
-
由 Aaron Patterson 提交于
The `anchor` parameter [is overridden](https://github.com/rails/rails/blob/b4b4a611d0eb9aa1c640c5f521c6a43bf2a65bab/actionpack/lib/action_dispatch/routing/mapper.rb#L1528) unless it is directly passed to `match`, so setting it in a scope must be a mistake.
-
- 13 8月, 2015 7 次提交
-
-
由 Aaron Patterson 提交于
this reduces the number of times we have to mutate the options hash.
-
由 Aaron Patterson 提交于
This just ensures that `format` is applied to things inside the scope
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
Eventually we don't want to expose the "options" hash from scope, only read values from it. Lets start by adding a reader method.
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
-
由 Aaron Patterson 提交于
We're going to try pulling this up further, and check `via` validity sooner. This way we don't have to do a bunch of processing on `options` hashes only to find out that the route is incorrect
-
- 12 8月, 2015 4 次提交
-
-
由 Aaron Patterson 提交于
this way we don't have to insert / delete it from the options hash so many times.
-
由 Dennis Suratna 提交于
-
由 Aaron Patterson 提交于
I think we can find the original place where `action` is added to the options hash now.
-
由 Aaron Patterson 提交于
we want to try to pull this logic up to where the user actually passed in "controller" so that it's close to the related call. That way when we're down the stack, we don't need to wonder "why are we doing this?"
-