no sortBy.DISPLAY_NAME:ASC

[KelvinC013]

this applies to the search_videos api call
I don't see a way to sort videos returned by display_name. Is this by design or something that just needs to be added? I've also noticed that when using this call the sortby parameter should display like so : sort_by=display_name:DESC . Is this handled by the wrapper?

Thanks

reference = http://support.brightcove.com/en/docs/searching-videos-media-api

[dmillz]

Hi ItsKelAgain,

It looks like the search_videos API call has changed since the .NET-MAPI-Wrapper was originally written.

The originally specified SortBy enum only included the fields shown here: http://docs.brightcove.com/en/media/#SortByType. It seems, though, that particular enum no longer applies to the search_videos call (which, to Brightcove's credit, is noted in the linked documentation). This is definitely a change from the API as it was specified when I first wrote the .NET wrapper.

Similarly, the special formatting required by search_videos for the sort_by parameter (ie. sort_by=fieldname:DIRECTION) is also new.

Currently the .NET wrapper doesn't correctly handle this case. I can try to find some time to fix it in the not-too-distant future; but it won't be for a few weeks, at minimum, due to my current workload and vacation plans.

If you can't wait, and if you're able to fork the project on Github and make the necessary changes yourself, I'd love to see any code you might be able to contribute back to the project. Since the existing SortBy enum still applies to other API calls, I would envision creating a new SearchVideosSortBy (or some similar name) enum (or class?) that is specific to the search_videos call, and then customizing the way it's serialized in order to meet the new requirements for this API call. It shouldn't be too hard :-)

I'm sorry the .NET wrapper doesn't currently handle this case, but I hope that the explanation helps. Please feel free to follow up if you have any additional questions.

Thanks!

Regards,
Dave

[KelvinC013]

Hello David,
I've forked the project and made the necessary changes.

btw - You may want to clean the code a bit before merging . Thanks

Regards,
Kel

On Wed, Aug 22, 2012 at 5:03 PM, David Mills notifications@github.comwrote:

Hi ItsKelAgain,

It looks like the search_videos API call has changed since the
.NET-MAPI-Wrapper was originally written.

The originally specified SortBy enum only included the fields shown here:
http://docs.brightcove.com/en/media/#SortByType. It seems, though, that
particular enum no longer applies to the search_videos call (which, to
Brightcove's credit, is noted in the linked documentation). This is
definitely a change from the API as it was specified when I first wrote the
.NET wrapper.

Similarly, the special formatting required by search_videos for the
sort_by parameter (ie. sort_by=fieldname:DIRECTION) is also new.

Currently the .NET wrapper doesn't correctly handle this case. I can try
to find some time to fix it in the not-too-distant future; but it won't be
for a few weeks, at minimum, due to my current workload and vacation plans.

If you can't wait, and if you're able to fork the project on Github and
make the necessary changes yourself, I'd love to see any code you might be
able to contribute back to the project. Since the existing SortBy enum
still applies to other API calls, I would envision creating a new
SearchVideosSortBy (or some similar name) enum (or class?) that is specific
to the search_videos call, and then customizing the way it's serialized in
order to meet the new requirements for this API call. It shouldn't be too
hard :-)

I'm sorry the .NET wrapper doesn't currently handle this case, but I hope
that the explanation helps. Please feel free to follow up if you have any
additional questions.

Thanks!

Regards,
Dave

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/13#issuecomment-7953602.

[dmillz]

Hi Kel,

I've just pushed out the 2.0 release. If you grab the latest dlls, the search_videos call should now function correctly, along with a handful of other fixes and improvements. See the changelog for full details: https://github.com/BrightcoveOS/.NET-MAPI-Wrapper/blob/master/CHANGELOG.markdown

I hope that helps!

Regards,
Dave