Skip to content

ActiveRecord::Base#new? should be #new_record? #120

Closed

Description

@catmando

deprecate new? and replace with new_record? which is the ActiveRecord method name.

Note: historically I think the name new? came in from perhaps Volt, as MongoMapper uses new?

Activity

  1. added this to the alpha1.4 milestone on Feb 15, 2019
  2. catmando commented on Feb 15, 2019

    @catmando
    ContributorAuthor

    needed to use new? for #117 so this got done with that.

  3. catmando commented on Feb 16, 2019

    @catmando
    ContributorAuthor

    I don't see any reason to deprecate. Its just an alias method

  4. Tim-Blokdijk commented on Apr 14, 2019

    @Tim-Blokdijk
    Contributor

    Seems to work, issue can be closed.
    Does not need explicit documentation, as it's a ActiveRecord method.

  5. modified the milestones: alpha1.4, alpha1.5 on Apr 18, 2019
  6. added
    ready-to-releaseInternal Use Only: Has been fixed, specs passing and pushed to edge branch
    on Apr 18, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    bugSomething isn't workingenhancementNew feature or requestready-to-releaseInternal Use Only: Has been fixed, specs passing and pushed to edge branch

    Type

    No type

    Projects

    No projects

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions