Partial cleanup of ManualInfo

src/main/java/tconstruct/tools/items/ManualInfo.java

@@ -9,54 +9,43 @@ import cpw.mods.fml.common.FMLCommonHandler;
 import cpw.mods.fml.relauncher.Side;
 import mantle.books.BookData;
 import mantle.books.BookDataStore;
+import tconstruct.TConstruct;
 import tconstruct.client.TProxyClient;
 
+/**
+ * This class is now just a constructor with side effects, so a glorified method call. TODO: Clean up when breaking API
+ * change is deemed acceptable.
+ */
 public class ManualInfo {
-    // static String[] name = new String[] { "beginner", "toolstation", "smeltery", "diary" };
-    // static String[] textureName = new String[] { "tinkerbook_diary", "tinkerbook_toolstation",
-    // "tinkerbook_smeltery", "tinkerbook_blue" };
-
-    BookData beginner = new BookData();
-    BookData toolStation = new BookData();
-    BookData smeltery = new BookData();
-    BookData diary = new BookData();
-    BookData weaponry = new BookData();
-
-    /*
-     * diary = readManual("/assets/tinker/manuals/diary.xml", dbFactory); volume1 =
-     * readManual("/assets/tinker/manuals/firstday.xml", dbFactory); volume2 =
-     * readManual("/assets/tinker/manuals/materials.xml", dbFactory); smelter =
-     * readManual("/assets/tinker/manuals/smeltery.xml", dbFactory);
-     */
 
     public ManualInfo() {
         Side side = FMLCommonHandler.instance().getEffectiveSide();
-        beginner = initManual(
-                beginner,
+        initManual(
+                new BookData(),
                 "tconstruct.manual.beginner",
                 "\u00a7o" + StatCollector.translateToLocal("manual1.tooltip"),
                 side == Side.CLIENT ? TProxyClient.volume1 : null,
                 "tinker:tinkerbook_diary");
-        toolStation = initManual(
-                toolStation,
+        initManual(
+                new BookData(),
                 "tconstruct.manual.toolstation",
                 "\u00a7o" + StatCollector.translateToLocal("manual2.tooltip"),
                 side == Side.CLIENT ? TProxyClient.volume2 : null,
                 "tinker:tinkerbook_toolstation");
-        smeltery = initManual(
-                smeltery,
+        initManual(
+                new BookData(),
                 "tconstruct.manual.smeltery",
                 "\u00a7o" + StatCollector.translateToLocal("manual3.tooltip"),
                 side == Side.CLIENT ? TProxyClient.smelter : null,
                 "tinker:tinkerbook_smeltery");
-        diary = initManual(
-                diary,
+        initManual(
+                new BookData(),
                 "tconstruct.manual.diary",
                 "\u00a7o" + StatCollector.translateToLocal("manual4.tooltip"),
                 side == Side.CLIENT ? TProxyClient.diary : null,
                 "tinker:tinkerbook_blue");
-        weaponry = initManual(
-                weaponry,
+        initManual(
+                new BookData(),
                 "tconstruct.manual.weaponry",
                 "\u00a7o" + StatCollector.translateToLocal("manual5.tooltip"),
                 side == Side.CLIENT ? TProxyClient.weaponry : null,
@@ -64,10 +53,9 @@ public class ManualInfo {
     }
 
     public BookData initManual(BookData data, String unlocName, String toolTip, Document xmlDoc, String itemImage) {
-        // proxy.readManuals();
         data.unlocalizedName = unlocName;
         data.toolTip = unlocName;
-        data.modID = "TConstruct";
+        data.modID = TConstruct.modID;
         data.itemImage = new ResourceLocation(data.modID, itemImage);
         data.doc = xmlDoc;
         BookDataStore.addBook(data);