[KodiProps] Fix multi wrapper separator

[knaerzche]

According to the wiki, the separator for multiple wrappers is ,, not +.

Description

ADP::KODI_PROPS::CCompKodiProps::ParseDrmOldProps currently builds multiple wrappers with + as seperator which makes it unparsable for HelperWv::TranslateWrapper .

Motivation and context

Fix a bug.

How has this been tested?

With a stream using json,base64 (JB)

Screenshots (if appropriate):

Types of change

• [ x] Bug fix (non-breaking change which fixes an issue)
• Clean up (non-breaking change which removes non-working, unmaintained functionality)
• Improvement (non-breaking change which improves existing functionality)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that will cause existing functionality to change)
• Cosmetic change (non-breaking change that doesn't touch code)
• None of the above (please explain below)

Checklist:

• [x ] I have read the Contributing document
• [ x] My code follows the Code Guidelines of this project
• My change requires a change to the Wiki documentation
• I have updated the documentation accordingly

[CastagnaIT]

this change is wrong,
as is also explained on wiki, the code its correct
i just forgot to fix the above method comment

[knaerzche]

It's not like I scanned the comments in this code and adapted the code to the comment: Playing a stream with the mentioned "unwrapping method" fails with: Cannot translate license wrapper, unknown type json+base64 and this change fixes it. I didn't look too deep in this code, but I'd guess it comes from here:

inputstream.adaptive/src/CompKodiProps.cpp

Line 520 in 8720557

drmCfg.license.unwrapper = "json+base64";

- it uses + as seperator, not ,

[CastagnaIT]

yeah good catch
please fix all them with ,, and also the above method comment

[knaerzche]

Done. (Took some time for me to find the wiki entry you were referring to)

[CastagnaIT]

well done
thank you for your first contrib