[mythtv-commits] Ticket #10420: Invalid data type in VideoLookup XSD

[MythTV]

#10420: Invalid data type in VideoLookup XSD
-------------------------------------------------+-------------------------
     Reporter:  Gregory Moyer <moyerg@…>         |      Owner:  dblain
         Type:  Patch - Bug Fix                  |     Status:  new
     Priority:  minor                            |  Milestone:  unknown
    Component:  Bindings - XML/JSON              |    Version:  Master Head
     Severity:  medium                           |   Keywords:
Ticket locked:  0                                |
-------------------------------------------------+-------------------------
 In the XSD generated for the VideoLookup data contract (used in the video
 service), there is a type of "xs:stringlist" for the "Countries" property
 that is not a valid W3C data type. It looks like QStringList is handled
 properly in wsdl.cpp, but there is an issue in xsd.cpp (which is why the
 WSDLs do not have this problem). The only data contract I found using
 QStringList was VideoLookup.

 The attached patch handles QStringList by turning it into ArrayOfString
 (as is done for the WSDLs currently). The reason the patch sets "sType" is
 because the line "typesToInclude.insert( sType, sContentType );" comes
 before the call to ConvertTypeToXSD. You may wish to rearrange things
 slightly by just overriding "sType" with ConvertTypeToXSD since it will
 turn "QStringList" into "ArrayOfString" if "bCustomType" is set (which it
 is now with this patch). My aim here was to be as minimally invasive as
 possible.

-- 
Ticket URL: <http://code.mythtv.org/trac/ticket/10420>
MythTV <http://code.mythtv.org/trac>
MythTV Media Center