Disable placeholders inside arguments by default + itemhook palceholder support fix

[BlitzOffline]

• Added per menu option arguments_support_placeholders that takes in a boolean value and defaults to false. When option is false, placeholders are not parsed inside user arguments.

• Changed how placeholders and arguments are parsed: arguments first (with placeholders inside value parsed if option mentioned above is true) and then placeholders

• Fixed placeholders not being parsed for itemhooks anymore.

[BlitzOffline]

Tested the following scenario:

with option enabled:

with option disabled:

[BlitzOffline]

Found possible issue with last change: if action is executed with delay, holder could not be valid anymore which is why it shouldn't actually be passed to ClickActionTask. This might be a bigger issue with delays that needs investigation

UPDATE: It seems that arguments were set on task initialization and not task execution because they are not available at execution. Placeholders however need to be set at execution time to make sure they return the most up to date value. I think the best solution that would sync actions with everything else is to provide arguments at execution time.

[BlitzOffline]

Fixed issues and retested!