Removing explicit cast where one isn't needed

As potion.getEffects() returns a list, we should just use the List object, and not cast to an ArrayList, as we risk a breakage in the future.