Fix Mixins issue

This was fun to solve \s

Author: Geolykt

src/main/java/de/geolykt/starloader/api/event/star/StarOwnershipTakeoverEvent.java

@@ -7,11 +7,8 @@
 import de.geolykt.starloader.api.event.Cancellable;
 
 /**
- * @deprecated Not fired by the API due to issues with mixins, other implementations are welcome to use it though
- *
  * Event fired when the ownership of a star changes.
  */
-@Deprecated(forRemoval = false)
 public class StarOwnershipTakeoverEvent extends StarEvent implements Cancellable {
 
     protected boolean cancelled = false;

src/main/java/de/geolykt/starloader/apimixins/EmpireMixins.java

@@ -416,7 +416,8 @@ public void setReligion(Religion religion) {
         a(religion);
     }
 
-    @Inject(method = "b", at = @At(value = "HEAD"), cancellable = true)
+// FIXME Mixin tends to behave a bit strange with this one
+    @Inject(method = "b(I)V", at = @At(value = "HEAD"), cancellable = true)
     public void setTechlevel(final int techLevel, final CallbackInfo ci) {
         if (techLevel == getTechnologyLevel()) {
             return;
@@ -446,7 +447,7 @@ public void tick(CallbackInfo info) {
     }
 
     @Shadow
-    public boolean Y() { // is larger empire
+    public boolean Y() { // isNoteable
        return false; // I am not sure that this is actually the name of the method, but I have to assume
        // that based on what I currently know about this method
     }

src/main/java/de/geolykt/starloader/apimixins/StarMixins.java

@@ -18,7 +18,9 @@
 import de.geolykt.starloader.api.NamespacedKey;
 import de.geolykt.starloader.api.empire.ActiveEmpire;
 import de.geolykt.starloader.api.empire.Star;
+import de.geolykt.starloader.api.event.EventManager;
 import de.geolykt.starloader.api.event.TickCallback;
+import de.geolykt.starloader.api.event.star.StarOwnershipTakeoverEvent;
 import snoddasmannen.galimulator.Religion;
 
 @Mixin(snoddasmannen.galimulator.Star.class)
@@ -249,19 +251,14 @@ public void syncCoordinates() {
         y = q.y;
     }
 
-/* FIXME Spongepowered's Mixins finds it very funny to not want to apply this mixin,
-         sadly I cannot do this otherwise (i. e. using @Overwrite instead of @Inject) since otherwise
-         I'd have a very angry snoddasmannen, and if I use some improper ASM hacks I'd have a very angry
-         userbase that is wondering why their classes load 10 times longer than they should be loading,
-         additionally I do not know the consequences of that action as improper ASM hacks do not stack well with Mixins.*/
-//    @Inject(method = "b", at = @At("HEAD"), cancellable = true)
-//    public void takeover(Empire empire, CallbackInfo info) {
-//        StarOwnershipTakeoverEvent event = new StarOwnershipTakeoverEvent(this, getAssignedEmpire(), (ActiveEmpire) empire);
-//        EventManager.handleEvent(event);
-//        if (event.isCancelled()) {
-//            info.cancel();
-//        }
-//    }
+    @Inject(method = "b(Lsnoddasmannen/galimulator/Empire;)V", at = @At("HEAD"), cancellable = true)
+    public void takeover(snoddasmannen.galimulator.Empire empire, CallbackInfo info) {
+        StarOwnershipTakeoverEvent event = new StarOwnershipTakeoverEvent(this, getAssignedEmpire(), (ActiveEmpire) empire);
+        EventManager.handleEvent(event);
+        if (event.isCancelled()) {
+            info.cancel();
+        }
+    }
 
     @Inject(method = "e", at = @At("HEAD"))
     public void tick(CallbackInfo info) {

src/main/resources/starloader-api.accesswidener

@@ -20,8 +20,3 @@ accessible      field   snoddasmannen/galimulator/Space                     i
 
 # Enums
 denumerised      class   snoddasmannen/galimulator/EmpireSpecial
-
-# Synthetics
-# Does not appear to work
-natural      field   snoddasmannen/galimulator/EmpireSpecial             r      [Lsnoddasmannen/galimulator/EmpireSpecial;
-accessible   field   snoddasmannen/galimulator/EmpireSpecial             r      [Lsnoddasmannen/galimulator/EmpireSpecial;