[Approved] Fix to "IMDB with AKA grabber" (1 Viewer)

Deda

Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    re: Fix to "IMDB with AKA grabber"

    Ok, I discarded all AKA things now Harry Potter or Spiderman returns all Films. If you have some more examples can you give me a search string so I can test it and see what's wrong.
     

    BalduinB

    Portal Member
    July 29, 2009
    36
    8
    Home Country
    Germany Germany
    re: Fix to "IMDB with AKA grabber"

    Well, but then it's not IMDB with AKA grabber anymore if you do so ;)
    The AKA is the most interesting thing about the script, so you can seach for other names than the original title.
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    re: Fix to "IMDB with AKA grabber"

    Ha ha I know, but then problem is:

    bool bSkipThisAkaEntry = imdbakaOptions.Contains("(TV)") |
    imdbakaOptions.Contains("(TV series)") |
    imdbakaOptions.Contains("(VG)") |
    imdbakaOptions.Contains("(poster title)") |
    imdbakaOptions.Contains("(working title)") |
    imdbakaOptions.Contains("(promotional abbreviation)") |
    imdbakaOptions.Contains("(video title)") |
    imdbakaOptions.Contains("(original script title)") |
    imdbakaOptions.Contains("(video box title)") |
    imdbakaOptions.Contains("(new title)") |
    imdbakaOptions.Contains("(DVD box title)");

    try to remove all entries with"title" then I think all will be in the list.

    I will try to do that later, I need to go out with kids for few hours then I'll be back :)
     

    BalduinB

    Portal Member
    July 29, 2009
    36
    8
    Home Country
    Germany Germany
    re: Fix to "IMDB with AKA grabber"

    I tried your version, with an AKA Title. The german name of "Avatar" is "Avatar - Aufbruch nach Pandora".
    When you try this title your version of the script doesn't come to an end. So it can't be the way to just leave out the AKA.

    Just as a hint, that we must regard that not everybody uses the original title.

    ... Working at it again ...
     

    Paranoid Delusion

    Moderation Manager
  • Premium Supporter
  • June 13, 2005
    13,062
    2,978
    Cheshire
    Home Country
    United Kingdom United Kingdom
    re: Fix to "IMDB with AKA grabber"

    Before Deda changes, notice scrollbar :(

    sshot-21.jpg

    and after

    sshot-26.jpg

    The above showed no info available.

    If the AKA script can be fixed great, but I'm also inclined to have seperate imdb.comEN one as a backup, this requires titles to be in English.

    The german name of "Avatar" is "Avatar - Aufbruch nach Pandora".

    BalduinB, don't get me wrong, but would expect you to the use the imdb.de_ofdb script for German titled films ;)

    Pretty accurate scrap, only one wrong film "Dr Strange (2007)" gives Dr Strangelove....

    :D again to you both
     

    BalduinB

    Portal Member
    July 29, 2009
    36
    8
    Home Country
    Germany Germany
    re: Fix to "IMDB with AKA grabber"

    BalduinB, don't get me wrong, but would expect you to the use the imdb.de_ofdb script for German titled films ;)

    Well ... but that's what the AKA-Search is for, isn't it?
    Not only german, but international titles.

    btw. I don't like the imdb.de_ofdb script. It works quite fine, but picks absolutely crappy pictures.
    You won't like these either ;)
    And I (personally) want english titles and descriptions. 98% of my movies are in english. I like to watch the original stuff, not the translated.

    @Deda: We have more work to do. ;)

    If after Writers is written "(WGA)" the script interprets this as writer, and not the writers listed after that.

    The Tagline has to fomats :(
    1. if there is only one tagline:
    <div class="info">
    <h5>Tagline:</h5>
    <div class="info-content">
    Enter the World
    </div>
    </div>

    2. if there are more taglines:
    <div class="info">

    <h5>Tagline:</h5>
    <div class="info-content">
    Blah, blah ... <a class="tn15more inline" href="/title/tt0082096/taglines" onClick="(new Image()).src='/rg/title-tease/taglines/images/b.gif?link=/title/tt0082096/taglines';">See more</a>&nbsp;&raquo;
    </div>
    </div>

    The old script works in the firtst case, but not in the second. My version works only in the second case (so far).

    Well, well ... at first I thought only plot, tagline and year are missing ... this gets more and more ;)
     

    Paranoid Delusion

    Moderation Manager
  • Premium Supporter
  • June 13, 2005
    13,062
    2,978
    Cheshire
    Home Country
    United Kingdom United Kingdom
    re: Fix to "IMDB with AKA grabber"

    Well ... but that's what the AKA-Search is for, isn't it?
    Not only german, but international titles

    I know, but not much use if it cannot get English ones right, especially as they tend to be the imdb default movie names, unless made in a non-english speaking country were the default language is used for title of course :)

    Keep up the fine tuning :D

    Just a quick question, how does MovingPictures cope with names like your Avatar film with German naming.
     

    DBeck

    Portal Member
    October 2, 2007
    14
    2
    Home Country
    England England
    re: Fix to "IMDB with AKA grabber"

    Just like to say thanks I am testing it now so far so good
     

    Deda

    Lead Dev MP1 Videos
  • Premium Supporter
  • March 18, 2009
    2,423
    2,385
    Zagreb
    Home Country
    Croatia Croatia
    re: Fix to "IMDB with AKA grabber"

    Ok I fixed tagline and writers (if there are more only first is displayed ->for now :) )

    It's still AKA without AKA but for testing is OK for me
     

    Paranoid Delusion

    Moderation Manager
  • Premium Supporter
  • June 13, 2005
    13,062
    2,978
    Cheshire
    Home Country
    United Kingdom United Kingdom
    re: Fix to "IMDB with AKA grabber"

    Ouch that was painful to check, good news is all taglines\writers seemed to be there, and no was not going to revert to older version to check against LOL
     

    Users who are viewing this thread

    Top Bottom