[Solved] 1.3.7 not saving logo URLs

[Tunes0710]

Hi all.
I'm a new user to EPG123. I downloaded 1.3.6.20 the day before 1.3.7 came out and quickly installed the update, but found an issue that I don't seem to be able to work around.

My primary use for EPG123 was to save SD listings to an XML to import in to xteve to use with my IPTV service for better listings.
An unexpected, but nice little bonus, was the fact EPG123 could link to SD for logos. Which meant, when I did a mapping in xteve, I could click update logo and it would pull it in from the URL.

This was working great on 1.3.6.20, however once I updated to 1.3.7, it wasn't linking to the SD logos anymore, but downloading them and pointing to them locally.
Just to make sure I'm giving as much info as possible. This is an example of the xml on 1.3.6.20

Code: Select all

  <channel id="EPG123.30644.schedulesdirect.org">
    <display-name>BBC1LDN</display-name>
    <display-name>BBC One London</display-name>
    <display-name>BBC1</display-name>
    <icon src="https://schedulesdirect-api20141201.s3.us-east-1.amazonaws.com/assets/stationLogos/s30644_white_360w_270h.png" width="360" height="270" />
  </channel>

And because I am acutely aware that it could be/probably is a PEBCAK issue here are grabs of the 1.3.6.20 configuration


And upon updating to 1.3.7, the config is showing as this



Now, if I click save and then close EPG123, when I re-open, the configuration stays the same.
However, if I click "Save & Execute", after it has run, when I open EPG123 the configuration has reverted to this

and the contents of the XML show this

Code: Select all

  <channel id="EPG123.30644.schedulesdirect.org">
    <display-name>BBC1LDN</display-name>
    <display-name>BBC One London</display-name>
    <display-name>BBC1</display-name>
    <icon src="C:\ProgramData\GaRyan2\epg123\logos\BBC1LDN.png" width="332" height="130" />
  </channel>

If I right click EPG123 and run as administrator, change the config to "use linked images", save and close EPG123, it will be ok when I reopen it, but once I click "Save & Execute", it will revert back to "use local images from .\logos folder" and the XML will point locally instead of online.

in case it matters, EPG123 is installed in to the default location of

Code: Select all

 "C:\Program Files (x86)\epg123"

and the output is in the default location of

Code: Select all

"C:\ProgramData\GaRyan2\epg123\output"

Am I being really stupid and missing a setting here? or is it a bug in EPG123?

Cheers


EDIT: Sorry... just wanted to add, that I've noticed there is an update (1.3.7.20). I updated to this, but the above problems still persist.

[adam1991]

Just go back to 1.3.6.20.

Problem solved.

cue the music: "if it ain't broke..."

[Tunes0710]

I have done so, but obviously wanted to report it in case it was a bug (rather than user error), so it can be found and fixed for future versions.
But little features like specifying the XML location which was implemented in 1.3.7 are "nice to haves", so if it can be made to work with this version, so much the better....

[garyan2]

@adam1991: you're assuming something is broken.

@Tunes0710: you just need to click the checkbox to 'Include station logos in .\logos folder' option. With the new option to identify preferred and alternate logos, this is what the code ended up requiring. I did think about linking the checkboxes together on a check, but went this route with putting a note on the XMLTV Tab.

[adam1991]

@adam1991: you're assuming something is broken.

No, I made *only* the point that 1.3.6 was NOT broken for him.

All we know is, he installed 1.3.6.20 and very soon thereafter saw a newer version and immediately installed it--despite his 1.3.6.20 working fine.

If 1.3.6.20 worked fine, and had all the features he wanted....if it ain't broke, don't fix it.

[garyan2]

Sorry, I read further and see you your settings may not be saving. I'll check that out.

[garyan2]

If the option to 'Include station logos...' is enabled, then it does maintain your setting to use the url, but I would say the behavior you see was unintended. Looks like one of my 'If...else' statements reverts to default for XMLTV if it is not enabled. You will be back to what you want by enabling the station logos on the configuration tab.

[garyan2]

Okay, found my logic error when the form first loads which will revert to "local". I'll get that fixed as soon as possible.

[garyan2]

Okay, pie on my face on this one. The XMLTV logo setting would revert to "local" on subsequent loads with a save. The logic has been fixed and v1.3.7.30 is released on the website.

@adam1991: Sorry... I would consider v1.3.7.20 to be broken for XMLTV logos. Point for adam1991!

[Tunes0710]

@adam1991: you're assuming something is broken.

@Tunes0710: you just need to click the checkbox to 'Include station logos in .\logos folder' option. With the new option to identify preferred and alternate logos, this is what the code ended up requiring. I did think about linking the checkboxes together on a check, but went this route with putting a note on the XMLTV Tab.

TOYlMNk.png

Aaah.. my apologies.
I took that message to mean that I only needed to check it if I wished to download the logos in to the .\logos folder and include them in the XML, and as I just wanted them to be linked to SD rather than actually downloaded, that is why I never checked that box.

I can confirm that after updating to .30, ticking the box and carrying out a Save&Execute, it now has the URLs in the XML and also keeps the settings after closing/re-opening.
Many thanks for sorting so quickly!

[garyan2]

No reason to apologise. This gives me insight into how folks interpret the tool and identifies problems with my code. I appreciate that a whole lot.

Thank You!