Small Fixes (#536)

* Small Fixes

* fix shift+v

* fix subset zindex

---------

Co-authored-by: slprime <history-21@yandex.ru>

Author: slprime

src/main/java/codechicken/nei/BookmarkPanel.java

@@ -130,9 +130,12 @@ public void setStartRowIndex(BookmarkGrid BGrid, int rowIndex) {
         }
 
         public void setEndRowIndex(BookmarkGrid BGrid, int rowIndex) {
-            this.endRowIndex = rowIndex;
-            this.endItemIndexTop = getTopItemIndex(BGrid, rowIndex);
-            this.endItemIndexBottom = getBottomItemIndex(BGrid, rowIndex);
+
+            if (BGrid.getRowItemIndex(rowIndex, true) >= 0) {
+                this.endRowIndex = rowIndex;
+                this.endItemIndexTop = getTopItemIndex(BGrid, rowIndex);
+                this.endItemIndexBottom = getBottomItemIndex(BGrid, rowIndex);
+            }
         }
 
         public boolean hasEndRow() {
@@ -226,12 +229,12 @@ public boolean equalsRecipe(BookmarkRecipeId recipeId, int groupId) {
         }
     }
 
-    public static enum BookmarkViewMode {
+    public enum BookmarkViewMode {
         DEFAULT,
         TODO_LIST
     }
 
-    public static enum BookmarkLoadingState {
+    public enum BookmarkLoadingState {
         LOADING,
         LOADED
     }
@@ -2726,8 +2729,9 @@ private ItemStack shiftStackSize(BookmarkGrid BGrid, int slotIndex, int shift, i
     }
 
     public boolean pullBookmarkItems(int groupId, boolean onlyIngredients) {
-        IBookmarkContainerHandler containerHandler = BookmarkContainerInfo
-                .getBookmarkContainerHandler(getGuiContainer());
+        final GuiContainer guiContainer = getGuiContainer();
+        final IBookmarkContainerHandler containerHandler = BookmarkContainerInfo
+                .getBookmarkContainerHandler(guiContainer);
 
         if (containerHandler == null) {
             return false;
@@ -2736,14 +2740,14 @@ public boolean pullBookmarkItems(int groupId, boolean onlyIngredients) {
         final BookmarkGrid BGrid = (BookmarkGrid) grid;
         final ArrayList<ItemStack> items = new ArrayList<>();
         final ItemStackMap<Long> uniqueItems = new ItemStackMap<>();
-        final BookmarkGroup group = groupId >= 0 ? BGrid.groups.get(groupId) : null;
         ItemStackMetadata meta;
 
         for (int idx = 0; idx < BGrid.realItems.size(); idx++) {
             meta = BGrid.metadata.get(idx);
 
             if (groupId == -1 || groupId == meta.groupId) {
                 final ItemStack stack = BGrid.getItem(idx);
+                final BookmarkGroup group = BGrid.groups.get(meta.groupId);
 
                 if (!onlyIngredients || meta.ingredient && (group == null || group.crafting == null
                         || group.crafting.inputs.containsKey(BGrid.realItems.get(idx)))) {
@@ -2768,7 +2772,7 @@ public boolean pullBookmarkItems(int groupId, boolean onlyIngredients) {
             return false;
         }
 
-        containerHandler.pullBookmarkItemsFromContainer(getGuiContainer(), items);
+        containerHandler.pullBookmarkItemsFromContainer(guiContainer, items);
         return true;
     }
 }

src/main/java/codechicken/nei/ItemZoom.java

@@ -61,6 +61,9 @@ public void draw(int mx, int my) {
 
             GL11.glPopMatrix();
 
+            GL11.glPushAttrib(GL11.GL_ALL_ATTRIB_BITS);
+            GuiContainerManager.enable2DRender();
+
             if (NEIClientConfig.getBooleanSetting("inventory.itemzoom.showName")) {
                 String dispalyName = NEIClientUtils.cropText(fontRenderer, this.displayName, this.availableAreaWidth);
                 drawStringC(
@@ -90,6 +93,7 @@ public void draw(int mx, int my) {
                 }
             }
 
+            GL11.glPopAttrib();
         }
     }
 

src/main/java/codechicken/nei/PresetsList.java

@@ -32,7 +32,7 @@
 
 public class PresetsList {
 
-    public static enum PresetMode {
+    public enum PresetMode {
         HIDE,
         REMOVE,
         GROUP;

src/main/java/codechicken/nei/SearchTokenParser.java

@@ -11,6 +11,7 @@
 
 import net.minecraft.client.Minecraft;
 import net.minecraft.client.resources.Language;
+import net.minecraft.item.ItemStack;
 import net.minecraft.util.EnumChatFormatting;
 
 import codechicken.nei.ItemList.AllMultiItemFilter;
@@ -69,6 +70,22 @@ public void clear() {
         }
     }
 
+    private static class IsRegisteredItemFilter implements ItemFilter {
+
+        public ItemFilter filter;
+
+        public IsRegisteredItemFilter(ItemFilter filter) {
+            this.filter = filter;
+        }
+
+        @Override
+        public boolean matches(ItemStack item) {
+            return item != null && item.getItem() != null
+                    && item.getItem().delegate.name() != null
+                    && this.filter.matches(item);
+        }
+    }
+
     protected final List<ISearchParserProvider> searchProviders;
     protected final ProvidersCache providersCache = new ProvidersCache();
     protected final TCharCharMap prefixRedefinitions = new TCharCharHashMap();
@@ -118,15 +135,15 @@ && getRedefinedPrefix(provider.getPrefix()) == ch)
 
     public ItemFilter getFilter(String filterText) {
         final String[] parts = EnumChatFormatting.getTextWithoutFormattingCodes(filterText).toLowerCase().split("\\|");
-        final List<ItemFilter> searchTokens = Arrays.stream(parts).map(s -> parseSearchText(s)).filter(s -> s != null)
+        final List<ItemFilter> searchTokens = Arrays.stream(parts).map(this::parseSearchText).filter(s -> s != null)
                 .collect(Collectors.toCollection(ArrayList::new));
 
         if (searchTokens.isEmpty()) {
             return new EverythingItemFilter();
         } else if (searchTokens.size() == 1) {
-            return searchTokens.get(0);
+            return new IsRegisteredItemFilter(searchTokens.get(0));
         } else {
-            return new AnyMultiItemFilter(searchTokens);
+            return new IsRegisteredItemFilter(new AnyMultiItemFilter(searchTokens));
         }
     }
 

src/main/java/codechicken/nei/SubsetWidget.java

@@ -22,6 +22,8 @@
 import net.minecraft.nbt.NBTTagList;
 import net.minecraft.util.EnumChatFormatting;
 
+import org.lwjgl.opengl.GL11;
+
 import codechicken.core.gui.GuiScrollSlot;
 import codechicken.lib.gui.GuiDraw;
 import codechicken.lib.vec.Rectangle4i;
@@ -711,9 +713,14 @@ public void draw(int mx, int my) {
 
         hoverStack = null;
         if (root.isVisible()) {
+            GL11.glPushAttrib(GL11.GL_ALL_ATTRIB_BITS);
+            GuiContainerManager.enable2DRender();
+
             root.resize(area.x, 0, area.y);
             root.cacheState();
             root.draw(mx, my);
+
+            GL11.glPopAttrib();
         }
     }