Luke 37118 Posted July 29, 2013 Share Posted July 29, 2013 Good, would also be nice to use only the ItemQuery model for all the item related queries. The problem with that is there will be a lot of additional params that are not supported by other calls which may cause confusion. Link to comment Share on other sites More sharing options...
Luke 37118 Posted July 29, 2013 Share Posted July 29, 2013 Ok, the first part of this is done. A bunch of methods have been removed, the ones that accept the BaseItemDto object : - GetGenreImageUrl - GetStudioImageUrl etc, etc. You can now pass the object directly into GetImageUrl and it will figure out which one to call based on the type. This way you don't have to have a lot of code that looks like this: - If Artist call GetArtistImageUrl - If Genre call GetGenreImageUrl etc. Of course, the methods that just accept the name of the item are still there, so if you want to do it that way, youc an. Link to comment Share on other sites More sharing options...
Armand 21 Posted July 30, 2013 Author Share Posted July 30, 2013 btw i saw that the MediaBrowser.Model.net35.dll was renamed to MediaBrowser.Model.dll for the net35 package. (which is fine) I didn't notice the change until i saw a fresh install fail because i tried to package the now non-existing net35.dll. Also if this was mentioned already somewhere else on this forum i have not said anything Link to comment Share on other sites More sharing options...
Luke 37118 Posted July 30, 2013 Share Posted July 30, 2013 That was Scott. He didn't mean to blindside you, I think he's just not aware of your development. Scott uses ApiClient for windows phone. He also works at MS and knows a lot about how nuget and pcl's work, so I think the change was more of a best practice type of thing. I'll make sure he sees this so that future changes are coordinated. Link to comment Share on other sites More sharing options...
ScottIsAFool 517 Posted July 31, 2013 Share Posted July 31, 2013 That was Scott. He didn't mean to blindside you, I think he's just not aware of your development. Scott uses ApiClient for windows phone. He also works at MS and knows a lot about how nuget and pcl's work, so I think the change was more of a best practice type of thing. I'll make sure he sees this so that future changes are coordinated. My bad Sorry about that. Link to comment Share on other sites More sharing options...
ebr 14935 Posted July 31, 2013 Share Posted July 31, 2013 btw i saw that the MediaBrowser.Model.net35.dll was renamed to MediaBrowser.Model.dll for the net35 package. (which is fine)I didn't notice the change until i saw a fresh install fail because i tried to package the now non-existing net35.dll. Also if this was mentioned already somewhere else on this forum i have not said anything That is going to blow up MBC as well... Link to comment Share on other sites More sharing options...
Luke 37118 Posted July 31, 2013 Share Posted July 31, 2013 Yea but not for existing users because they'll be on the last nuget package. You just have to fix the references the next time you pull it down. Link to comment Share on other sites More sharing options...
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now