[aidden]

Inventory.newQuery().names("Guam potion (unf)").results().first() returns null
Inventory.newQuery().names("Guam potion \\(unf\\)").results().first() finds the correct item

@Cloud

 

[slashnhax]

Seems like the names(String...) is converting the Strings to Patterns but doesn't escape the brackets :/

Edit: Which means this might be occurring in the other queries as well

 

[cloud]

For the next release, does this check out to you?

public static Pattern getPatternForExactString(final String string) {
String pattern = "^";
for (char c : string.toCharArray()) {
if (c == '(' || c == ')' || c == '^' || c == '$'|| c == '.' || c == '*' || c == '?' || c == '|'|| c == '[' || c == '{') {
pattern += "\\";
}
pattern += c;
}
pattern += "$";
return Pattern.compile(pattern);
}

 

[slashnhax]

For the next release, does this check out to you?

public static Pattern getPatternForExactString(final String string) {
String pattern = "^";
for (char c : string.toCharArray()) {
if (c == '(' || c == ')' || c == '^' || c == '$'|| c == '.' || c == '*' || c == '?' || c == '|'|| c == '[' || c == '{') {
pattern += "\\";
}
pattern += c;
}
pattern += "$";
return Pattern.compile(pattern);
}

What about the closing braces and brackets?

 

[viewer]

You could use IDs instead of names anyway since SpriteItem IDs dont change

 

[aidden]

You could use IDs instead of names anyway since SpriteItem IDs dont change

While that's true, it's could cause problems in other areas.

 

[slashnhax]

While that's true, it's could cause problems in other areas.

Plus you'd have to gather all of the IDs, which would be a drag

 

[cloud]

What about the closing braces and brackets?

From my research, those only need to be escaped when the prior opening isn't escaped.

You could use IDs instead of names anyway since SpriteItem IDs dont change

Yeah, but I really don't recommend that.