Athena Skin Missing Screenshots (1 Viewer)

SpudR

Documentation Group
  • Team MediaPortal
  • July 27, 2007
    2,632
    702
    Yorkshire, UK
    England England
    Country flag
    Pressing the 'Screenshot' button on the MPEI > Known Extensions > Athena shows an image placeholder, then crashes the MPEI totally, if you step through the 5.
     

    JimCatMP

    Documentation Group
  • Team MediaPortal
  • April 1, 2010
    648
    281
    Leeds
    United Kingdom United Kingdom
    Country flag
    Not much to say other than 'confirmed' :oops:

    Testing a few other, many missing images [Xtreme - 32 listed, none present:rolleyes:, only Athena seems to crash MPEI on ones I tested [5 or 6].

    TTFN - JCMP
     

    Edalex

    Community Plugin Dev
  • Premium Supporter
  • January 3, 2008
    2,922
    1,228
    Saratov
    Russian Federation Russian Federation
    Country flag
    I've looked at Athena MPEI.
    You saw placeholders instead of images because links are 404
    XML:
    <SectionParam Name="Online Screenshots">
    <Value>
    http://www.team-mediaportal.com/components/com_mtree/img/listings/m/2304.jpg; http://www.team-mediaportal.com/components/com_mtree/img/listings/m/2307.jpg; http://www.team-mediaportal.com/components/com_mtree/img/listings/m/2306.jpg; http://www.team-mediaportal.com/components/com_mtree/img/listings/m/2305.jpg;
    </Value>
    <ValueType>String</ValueType>
    <Description>Online stored screenshot urls separated by ;</Description>
    </SectionParam>
    Current images on site look like this https://www.team-mediaportal.com/media/com_mtree/images/listings/o/2307.jpg
    So probably site changed a bit.
    I can't say for sure about crash but looks like MPEI thinks thre are 5 pictures. Obviously, because of additional ; after the text.
    I didn't find the code executing Screenshot loading by github search and I'm lazy to debug MPEI so you could believe in my vesion or wait another one. ;)
    Nah, I found it MediaPortal-1/ScreenShotNavigator.cs at ff4edb5f3612e397136971c4a3a57a2aa2ffcfc8 · MediaPortal/MediaPortal-1 · GitHub
    No checks if urls are valid there.
    And yes, such exceptions should be handled properly to avoid crashes and yes, MPEI should have log.
    Br.
     
    Last edited:

    high

    Administrator
  • Team MediaPortal
  • August 31, 2004
    4,928
    5,066
    Isselburg/NRW
    Germany Germany
    Country flag
    image-links changed ~ 2 years ago. I can create a rewrite for these old ones but still, there will be missing images cause author deleted/exchanged them. In athena for example 2304.jpg.

    false urls need to be covered in MPEI-code properly. I can fix local xml files like in the Athena-case but not external hosted ones (github/Google Code)
     
    Last edited:

    high

    Administrator
  • Team MediaPortal
  • August 31, 2004
    4,928
    5,066
    Isselburg/NRW
    Germany Germany
    Country flag
    Athena is fixed, the issue should not show up again for this item .
    Also rewrite old images to 'new' location to fix old ones like Xtreme
     

    Edalex

    Community Plugin Dev
  • Premium Supporter
  • January 3, 2008
    2,922
    1,228
    Saratov
    Russian Federation Russian Federation
    Country flag
    not that much I can do, image-links changed ~ 2 years ago. I can create a rewrite for these old ones but still, there will be missing images cause author deleted/exchanged them. In athena for example 2304.jpg.

    This needs to be fixed in MPEI-code properly.
    What about redirection from old path to new? Not sure Mpei will handle redirect properly though :cautious:
    Upd. Ooops, I didn't read your second message :D
     

    Users Who Are Viewing This Thread (Users: 0, Guests: 1)

    OP Similar threads Forum Replies Date
    Rob Hexenmeister Quick question - skinning MyPics General Support 7
    ajs [Info] Pre 0.7 Beta - MPExtended, WebMP, Bootstrap skin by ajs - Hard way MPExtended 296
    S Touchscreen skin? MediaPortal 1 Talk 6
    azzuro [fixed] 5011: Version label in Splash screen - pos, size from skin MediaPortal 1 61
    catavolt [WiP] When changing theme in my skins, I always get this error message in logs MediaPortal 1 9
    joecrow MultiShortcut Plugin (Uavailable) in DWHD skin General Support 58
    V Skin With Menu Editor MediaPortal 1 Skins 13
    wizard123 New Skin Template ? MediaPortal 2 8
    Rob Hexenmeister [solved] Latest Media Handler - skinning buttons Fanart Handler 19
    CyberSimian [solved] MP1-4976 Built-in skin variables for current date MediaPortal 1 8
    Mr6686 [solved] Titan skin broken since update to 1.23 Installation, configuration support 6
    doskabouter Skin question Skins and Plugins Installer (MPEI) 1
    Y Titan Skin unter 1.23 nicht mehr so schön Allgemeines Supportforum 1
    framug 1.22.0 BasicHome Editor settings for DefaultWide HD skin not kept when update 1.22 3
    catavolt [solved] FanartHandler - Skin - Holiday and Spotlight support Fanart Handler 5
    ajs [fixed] 4936 - Strange behavior of the font in the skin MediaPortal 1 35
    ajs [solved] 4935: Add string.replace function to Skin engine MediaPortal 1 10
    ajs [solved] Latest Media Handler - Skin settings - Custom latests items number Fanart Handler 27
    Rob Hexenmeister [solved] Latests Media Handler - Skinning an updating window or control Fanart Handler 26
    T Skin Wechsel Installationsprobleme? 1
    JJDoherty Designing & implementing a new skin... Skins and Design 8
    D [fixed] Image browser crashing in Folder View (WMC skin) 2.2 Archive 10
    CyberSimian Do skin variables have global scope? MediaPortal 1 16
    N Skins to download separately MediaPortal 1 Talk 5
    P WMC Skin full screen/EPG Skins and Design 3
    Similar threads

























    Top Bottom