INTERACT FORUM

Please login or register.

Login with username, password and session length
Advanced search  
Pages: [1]   Go Down

Author Topic: Problem with Date tagging syntax via MCWS SetInfo command  (Read 3071 times)

AndrewFG

  • MC Beta Team
  • Citizen of the Universe
  • *****
  • Posts: 3392
Problem with Date tagging syntax via MCWS SetInfo command
« on: August 28, 2016, 09:55:49 am »

There is an issue when you use the MCWS SetInfo command to modify date tags on a file.

The syntax of the MCWS call is as follows (where "dddd" is supposed to be a date value encoded as a large integer)..

GET /MCWS/v1/File/SetInfo?File=123&Field=Date%20Created&value=dddd HTTP/1.1

Now there are some control points out there (even ones belonging the JRiver) that do not encode the date "dddd" as a large integer, but rather encode it as a plain text date, such as "6/12/2016" or "2016-06-12" etc. being the plain text date that the user types in. e.g. as follows..

GET /MCWS/v1/File/SetInfo?File=123&Field=Date%20Created&value=6/12/2016 HTTP/1.1

The syntax shown above fails. So my request is that you kindly extend the MC MCWS server SetInfo syntax to allow the existing encoding syntax, plus also an extended syntax allowing plain text date values as well. i.e. So that the MCWS server will parse and convert "6/12/2016" or "2016-06-12" or similar plain text date values, to MC internal date values.

Obviously you could insist that every control point must deliver the proper long integer date encoding, but I think it would provide much more value if you extended the SetInfo syntax on the MC MCWS server side. This makes it easier for external users to write add ons.

( Note that I posted a bug report recently in this forum but Henrik moved it over to Remotes, and we all know that over there, the bug will suffer a long and painful death by neglect. However if you extend the MCWS server syntax as requested, then that bug will be fixed without you having to touch JRemote. http://yabb.jriver.com/interact/index.php/topic,106690.0.html )



Logged
Author of Whitebear Digital Media Renderer Analyser - http://www.whitebear.ch/dmra.htm
Author of Whitebear - http://www.whitebear.ch/mediaserver.htm

Matt

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 42384
  • Shoes gone again!
Re: Problem with Date tagging syntax via MCWS SetInfo command
« Reply #1 on: August 29, 2016, 11:25:11 am »

Coming next build:
Changed: Made the function File/SetInfo take a parameter "Formatted" that when set to one the value coming in is formatted (like a date).  Defaults to "1".

This way the date format that we expect will now be the "formatted" friendly style (instead of a wonky internally formatted number).
Logged
Matt Ashland, JRiver Media Center

AndrewFG

  • MC Beta Team
  • Citizen of the Universe
  • *****
  • Posts: 3392
Re: Problem with Date tagging syntax via MCWS SetInfo command
« Reply #2 on: August 29, 2016, 12:09:57 pm »

Brilliant!!
Logged
Author of Whitebear Digital Media Renderer Analyser - http://www.whitebear.ch/dmra.htm
Author of Whitebear - http://www.whitebear.ch/mediaserver.htm

jmone

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 14465
  • I won! I won!
Re: Problem with Date tagging syntax via MCWS SetInfo command
« Reply #3 on: August 29, 2016, 03:57:48 pm »

Nice!  Will the same apply for GetInfo and MPL export/imports as well?
Thanks
Nathan
Logged
JRiver CEO Elect

AndrewFG

  • MC Beta Team
  • Citizen of the Universe
  • *****
  • Posts: 3392
Re: Problem with Date tagging syntax via MCWS SetInfo command
« Reply #4 on: August 30, 2016, 12:24:12 am »

Will the same apply for GetInfo and MPL export/imports as well?

It is a good idea, but be careful changing GetInfo because it would break JRemote, (and possibly Gizmo too)..
Logged
Author of Whitebear Digital Media Renderer Analyser - http://www.whitebear.ch/dmra.htm
Author of Whitebear - http://www.whitebear.ch/mediaserver.htm

jmone

  • Administrator
  • Citizen of the Universe
  • *****
  • Posts: 14465
  • I won! I won!
Re: Problem with Date tagging syntax via MCWS SetInfo command
« Reply #5 on: August 30, 2016, 01:03:26 am »

....if the parameter is optional?...it  may be backward compatible.   

Also potentially impacted for the new date syntax changes would be Files/Current, and Files/Selected
Logged
JRiver CEO Elect
Pages: [1]   Go Up