Backend: Fishing Hook Display Stop using onTick

[AverageUser125]

Dependencies

• Backend: Text Display Update Events #6202

What

Instead of doing a check each tick, it will see all custom name changes and create the display/renderable once.
If it's there is not known entity it uses the detected one,
if it's the same entity as the previous it just updates,
and if it's a different entity it uses the one closest to the bobber.

I have tested in torrhus, and it works.
(Some users report that it currently randomly stops showing the display like 1h after fishing, but I can't replicate that or prove this fixed it. but I hope the "closest to bobber" logic maybe fixes that)

Changelog Technical Details

• Improved FishingHookDisplay to only run on entity custom name change instead of each tick. - Avrg

[github-actions]

Dependencies

This PR is now waiting on the following dependency:

• Backend: Text Display Update Events #6202

[github-actions]

Detekt found 1 issue

Show previous warnings

⚠️ Detekt found 1 issue ⚠️

• CheckRenderEntityEvent.kt:4: Imports must be ordered in lexicographic order with "java", "javax", "kotlin", "kotlinx" and aliases in the end.

More Details

• CheckRenderEntityEvent.kt:4
  message: Imports must be ordered in lexicographic order with "java", "javax", "kotlin", "kotlinx" and aliases in the end.
  rule: detekt.ImportRules.CustomImportOrdering
  path: src/main/java/at/hannibal2/skyhanni/events/CheckRenderEntityEvent.kt