search

mothership-ec / cog-user

User Cogule
Other
0 stars 1 forks source link

BC BREAK: Remove Create::save() method #28

Open joeholdcroft opened 11 years ago

joeholdcroft commented 11 years ago

It was mistakenly named save() rather than create(). We left it in for BC. In the next major release, remove this method.

This will require changes in the Mothership\User and Mothership\Ecommerce modules, and possibly others.

irisSchaffer commented 11 years ago

https://github.com/messagedigital/cog-mothership-user/commit/9dde5eae0eb11d2b34a1df0c007926a72f2235c4 and https://github.com/messagedigital/cog-mothership-ecommerce/commit/2e6c833ee177821d9d0ac20f836c4f8b240ab120

I searched uniform_wares_ms with all the vendor folders and those two were the only two places where I found ('user.create')->. Also went through all search results for save(.

joeholdcroft commented 11 years ago

Great, thanks. Then this can now be done.