Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

RPTools_Map_Logo.png is being downloaded instead of part of package #920

Closed
Azhrei opened this issue Nov 18, 2019 · 3 comments
Closed

RPTools_Map_Logo.png is being downloaded instead of part of package #920

Azhrei opened this issue Nov 18, 2019 · 3 comments
Assignees
Labels
bug tested This issue has been QA tested by someone other than the developer.

Comments

@Azhrei
Copy link
Member

Azhrei commented Nov 18, 2019

Describe the bug
In OSXAdapter, the image used for the macOS dock icon is being downloaded dynamically from RPTools.net and then cached for future use. It's using an http:// scheme that seems to fail in Eclipse (but works elsewhere), but we should be populating the JAR with this file instead and referencing it from there.

To Reproduce
Steps to reproduce the behavior:

  1. Start MapTool within Eclipse on macOS
  2. Note the exception from the PNG not being downloaded and the dock having the generic "Java" logo

Expected behavior
With the logo embedded, the image should always be available.

MapTool Info

  • Version: latest develop branch (1.5.7)
  • Install: None (Eclipse build)

Desktop (please complete the following information):

  • OS: macOS only
  • Version macOS Mojave 10.14.6 (although I believe not OS version dependent)

Additional context
No idea why this wasn't embedded into the JAR years ago. :(

Once cached, the name is referred to as maptool-dock-icon.png so we should probably just name it that in the JAR and refer to it the same way...

@Phergus Phergus added the bug label Nov 19, 2019
xdy added a commit to xdy-forks/maptool that referenced this issue Dec 8, 2019
xdy added a commit to xdy-forks/maptool that referenced this issue Dec 8, 2019
xdy added a commit to xdy-forks/maptool that referenced this issue Dec 8, 2019
@xdy
Copy link
Collaborator

xdy commented Dec 8, 2019

Just made a PR, #977, that might fix #920, I can't check as I don't have a mac...

Azhrei added a commit that referenced this issue Dec 14, 2019
#920 Don't download mac dock icon
@Phergus
Copy link
Contributor

Phergus commented Dec 15, 2019

@Azhrei Have you tested the fix?

@Azhrei
Copy link
Member Author

Azhrei commented Dec 15, 2019

Okay, tested from within Eclipse and it works...

And now I've built an uberJar and that works as well.

I'll close it.

@Azhrei Azhrei closed this as completed Dec 15, 2019
@Phergus Phergus added the tested This issue has been QA tested by someone other than the developer. label Dec 15, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug tested This issue has been QA tested by someone other than the developer.
Projects
None yet
Development

No branches or pull requests

3 participants