Fix param added with empty query ignores options

[gabrieldutra]

What type of PR is this? (check all applicable)

• Bug Fix

Description

This one was a bit hard to find 😅. From what I understood on this code:

redash/client/app/services/query.js

Lines 226 to 235 in a682265

	if (this.query.query && this.query.query === this.cachedQueryText) {
	const parameters = this.query.options.parameters;
	const hasUnprocessedParameters = find(parameters, p => !(p instanceof Parameter));
	if (hasUnprocessedParameters) {
	this.query.options.parameters = map(parameters, p =>
	p instanceof Parameter ? p : createParameter(p, this.query.id)
	);
	}
	return;
	}

It exists to determine when to skip parameters sync (query text with actual parameters list in query object). Since parameters are added in a context of inequality between the cachedQuery and the query text, the idea is to keep those parameters in there until the query text is modified (what wasn't happening for an empty query text)

Related Tickets & Documents

Fixes #4723
and #4146 is the one that triggered this context to be possible

Mobile & Desktop Screenshots/Recordings (if there are UI changes)

--