Skip to content

Update temple-osrs#4691

Merged
Nightfirecat merged 3 commits into
runelite:masterfrom
SMaloney2017:master
Aug 6, 2023
Merged

Update temple-osrs#4691
Nightfirecat merged 3 commits into
runelite:masterfrom
SMaloney2017:master

Conversation

@SMaloney2017
Copy link
Copy Markdown
Contributor

DT2 Bosses, Error Catching for missing Icons

@runelite-github-app
Copy link
Copy Markdown

runelite-github-app Bot commented Jul 28, 2023

@Adam- Adam- closed this Jul 28, 2023
@Adam- Adam- reopened this Jul 28, 2023
@Adam-
Copy link
Copy Markdown
Member

Adam- commented Jul 28, 2023

You have a gif with a png file extension which breaks our code scanner. Don't do this.

src/main/resources/com/templeosrs/bosses/game_icon_ironehb.png:                      GIF image data, version 89a, 16 x 16

Copy link
Copy Markdown
Member

@Adam- Adam- left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fails scanner

Changed rouge gif to png
@SMaloney2017
Copy link
Copy Markdown
Contributor Author

Fixed that weird GIF, but I'm not seeing any scanner output in the build details so I hope that was the only one...

@SMaloney2017 SMaloney2017 requested a review from Adam- August 4, 2023 20:27
@Nightfirecat
Copy link
Copy Markdown
Member

The scanner output is not public. Also you have other images which have the same problem--namely the below files:

$ git ls-files *.png | xargs file | grep -E 'png:\s+GIF'
src/main/resources/com/templeosrs/overall.png:                                       GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_agility.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_attack.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_construction.png:                GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_cooking.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_crafting.png:                    GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_defence.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_ehp.png:                         GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_f2pehp.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_fantasyehp.png:                  GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_farming.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_firemaking.png:                  GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_fishing.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_fletching.png:                   GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_herblore.png:                    GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_hitpoints.png:                   GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_hunter.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_ironehp.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_lvl3ehp.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_magic.png:                       GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_mining.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_overall.png:                     GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_prayer.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_ranged.png:                      GIF image data, version 89a, 18 x 18
src/main/resources/com/templeosrs/skills/skill_icon_runecraft.png:                   GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_slayer.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_smithing.png:                    GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_strength.png:                    GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_thieving.png:                    GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_uimehp.png:                      GIF image data, version 89a, 16 x 16
src/main/resources/com/templeosrs/skills/skill_icon_woodcutting.png:                 GIF image data, version 89a, 16 x 16

@SMaloney2017
Copy link
Copy Markdown
Contributor Author

I see, I suppose I've just been saving images from the OSRS HiScores so I had no idea. I appreciate the insight, I'll get these fixed.

Fixing all broken PNGs
@SMaloney2017
Copy link
Copy Markdown
Contributor Author

Alright, all should have been replaced. Sorry about that scanner trouble, shouldn't be an issue from here on out.

@Nightfirecat Nightfirecat merged commit 9f9c021 into runelite:master Aug 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants