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 #57

Closed
wants to merge 6 commits into from
Closed

Active Resource should warn about missing prefix value #57

wants to merge 6 commits into from

Commits on Sep 27, 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 27, 2010
    Configuration menu
    Copy the full SHA
    a71e07d 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 27, 2010
    Configuration menu
    Copy the full SHA
    30fb363 View commit details
    Browse the repository at this point in the history

  3. - better name for prefix param test case

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

  4. - check prefix options in collection_path

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

  5. - update exceptions documentation

    @gramos
    gramos committed Sep 27, 2010
    Configuration menu
    Copy the full SHA
    e405dbc 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 27, 2010
    Configuration menu
    Copy the full SHA
    f85b38a View commit details
    Browse the repository at this point in the history