Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Active Resource should warn about missing prefix value #37

Closed
wants to merge 7 commits into from
Closed

Active Resource should warn about missing prefix value #37

wants to merge 7 commits into from

Commits on Sep 18, 2010

  1. - elmenth_path raise an ActiveResource::MissingPrefixParam exception … 

    …when prefix_options does not has all required prefix_options ex: class StreetAddress < ActiveResource::Base self.site = "http://37s.sunrise.i:3000/people/:person_id/" end

    StreetAddress.element_path(1)
    # => ActiveResource::MissingPrefixParam
    @gramos
    gramos committed Sep 18, 2010
    Configuration menu
    Copy the full SHA
    9309d8c View commit details
    Browse the repository at this point in the history

  2. - refactoring, move prefix_options check to a custom method

    @gramos
    gramos committed Sep 18, 2010
    Configuration menu
    Copy the full SHA
    ec13ce9 View commit details
    Browse the repository at this point in the history

  3. - better name for prefix param test case

    @gramos
    gramos committed Sep 18, 2010
    Configuration menu
    Copy the full SHA
    d95ae73 View commit details
    Browse the repository at this point in the history

  4. - check prefix options in collection_path

    @gramos
    gramos committed Sep 18, 2010
    Configuration menu
    Copy the full SHA
    37979de View commit details
    Browse the repository at this point in the history

  5. - update exceptions documentation

    @gramos
    gramos committed Sep 18, 2010
    Configuration menu
    Copy the full SHA
    759c2e2 View commit details
    Browse the repository at this point in the history

  6. - added mock to test ActiveResource::MissingPrefixParam in finder_test

    @gramos
    gramos committed Sep 18, 2010
    Configuration menu
    Copy the full SHA
    f7c196a View commit details
    Browse the repository at this point in the history

Commits on Sep 20, 2010

  1. Merge remote branch 'upstream/master'

    @gramos
    gramos committed Sep 20, 2010
    Configuration menu
    Copy the full SHA
    8f3311e View commit details
    Browse the repository at this point in the history