Skip to content

[CLEANUP beta] Remove metaPath, getMeta and setMeta#11750

Merged
krisselden merged 1 commit intoemberjs:masterfrom
duggiefresh:cleanup-remove-metaPath
Jul 19, 2015
Merged

[CLEANUP beta] Remove metaPath, getMeta and setMeta#11750
krisselden merged 1 commit intoemberjs:masterfrom
duggiefresh:cleanup-remove-metaPath

Conversation

@duggiefresh
Copy link
Contributor

metaPath had been deprecated. setMeta and getMeta were not
being used at all... must of been leftover cruft.

`metaPath` had been deprecated. `setMeta` and `getMeta` were not
being used at all... must of been leftover cruft.
@duggiefresh duggiefresh force-pushed the cleanup-remove-metaPath branch from 7226e15 to 7eb9eb3 Compare July 15, 2015 03:57
@stefanpenner
Copy link
Member

I believe it was just used by the old ac/rc code

@stefanpenner
Copy link
Member

Get get and set meta where exposed and not deprecated. We may need to deprecate them?

Thoughts @rwjblue ?

@mmun
Copy link
Member

mmun commented Jul 15, 2015

@stefanpenner Speaking of which, we can remove

this.cacheMeta = undefined;
now.

@krisselden
Copy link
Contributor

These were internal, used briefly by Ember Data, but have been unused a while.

krisselden added a commit that referenced this pull request Jul 19, 2015
[CLEANUP beta] Remove `metaPath`, `getMeta` and `setMeta`
@krisselden krisselden merged commit ed526d0 into emberjs:master Jul 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants