[WebService] General Media Access Webservice (2 Viewers)

Mike Lowrey

Portal Pro
February 4, 2009
638
124
Berlin
Home Country
Germany Germany
The problem is Microsoft uses a somehow more complex WSDL file(with more layers) which isn't always perfectly consumable by any generator.

That's the main reason why we also added a json endpoint which nearly anyone can use without much hassle.
 

nip.nyborg

Portal Pro
January 12, 2007
98
9
Home Country
Denmark Denmark
Hi Mike
I tried the new version, but I can´t get to work, when I run the test link, I get “endpoint not found”.
(with and without firewall on)
Any suggestions
 

Oxan

Retired Team Member
  • Premium Supporter
  • August 29, 2009
    1,730
    1,124
    Home Country
    Netherlands Netherlands
    Currently there's a lot of code duplication in the web service. Most methods open the database, execute a query, loop through the results and select a subset of it (_start and _end), and close the database. This isn't very good for code readability. I've created an experimental patch for this: it creates a Database class, which handles the standard method calls. I've been able to reduce the size of MPTvSeries.cs with almost 40%, but more importantly, the code is a lot cleaner (for example removal of the -99 magic number). It shouldn't have any effect on users as the API isn't changed, but only the internal code. I can cleanup the other classes also, but the patch is already quite big and I want to avoid to create an monster-patch which is un-updatable (is that even a word?).

    Mike: the WebMediaPortal patches will come soon too, working on it now :)
     

    Attachments

    • gma-cleanup.patch
      36.1 KB

    DieBagger

    Retired Team Member
  • Premium Supporter
  • September 11, 2007
    2,516
    1,276
    40
    Austria
    Home Country
    Austria Austria
    Currently there's a lot of code duplication in the web service. Most methods open the database, execute a query, loop through the results and select a subset of it (_start and _end), and close the database. This isn't very good for code readability. I've created an experimental patch for this: it creates a Database class, which handles the standard method calls. I've been able to reduce the size of MPTvSeries.cs with almost 40%, but more importantly, the code is a lot cleaner (for example removal of the -99 magic number). It shouldn't have any effect on users as the API isn't changed, but only the internal code. I can cleanup the other classes also, but the patch is already quite big and I want to avoid to create an monster-patch which is un-updatable (is that even a word?).

    Mike: the WebMediaPortal patches will come soon too, working on it now :)

    Hi Oxan,

    I'm (and afaik Mike Lowrey as well) using TFS for the codeplex project (not svn) so afaik it's not really possible to apply your patches for me (us).

    @Mike Lowrey: What the best way to provide patches in tfs?

    :D
     

    Mike Lowrey

    Portal Pro
    February 4, 2009
    638
    124
    Berlin
    Home Country
    Germany Germany
    Codeplex has it's own client for patches. But since Codeplex also supports SVN it should be no problem to use the patch with this kind of access.
     

    Mike Lowrey

    Portal Pro
    February 4, 2009
    638
    124
    Berlin
    Home Country
    Germany Germany
    Hm may be i'm doing something wrong but i'm unable to apply the patchfile on a svn checkout...
    May be you can just send me the corresponding cs files ;)


    edit:
    Or if you have a codeplex account i just give you access to the repository...
     

    Oxan

    Retired Team Member
  • Premium Supporter
  • August 29, 2009
    1,730
    1,124
    Home Country
    Netherlands Netherlands
    Hm may be i'm doing something wrong but i'm unable to apply the patchfile on a svn checkout...
    May be you can just send me the corresponding cs files ;)


    edit:
    Or if you have a codeplex account i just give you access to the repository...
    Oh, I see, I've created the patch a directory too low... Stupid incompatibility. Anyway, I've attached the .cs files. My codeplex account is oxan.
     

    Attachments

    • GMA-patched-files.zip
      4.9 KB

    Users who are viewing this thread

    Top Bottom