Please do the following:

• there is dedicated kApplication::unescapeRequestVariable method that needs to be used in cases, where on Front-End we need to revert force htmlspecialchars call (don't use kUtil::unescape for that)
• use kUtil::escape with ...ESCAPE_URL argument instead of direct call to urlencode

There is no point in using " because it's not used in any other HTML tag attribute value. So please use ".

Don't we have any tag like SearchLink that builds link to template and passes keywords to it?

Return back.

Undo changes to this method.

Undo changes to this method.

Not done.

1. The $request_keywords contains " when " was present in search keywords (e.g. test " link), but $keywords properly has " and they never match resulting no search performed. Please use $this->Application->unescapeRequestVariable method to avoid this.
   
2. Please do a trim of keywords before comparing because the $keywords is always trimmed and when we have trailing spaces then we'll have a non-match all the time.

I've performed one search and then I'm getting it's result every time I do a next search. I think that conditions used are not quite accurate:

!$request_keywords

This will match 2 things:

• absent keyword
• present, but empty keyword

If later is expected behavior then use === false instead of !

$request_keywords == $session_keywords

This sounds like correct one (new keywords match to session = reuse previous search).

!$this->Application->GetVar('do_not_drop_search_table')

The do_not_drop_search_table request variable is set by 1st OnSimpleSearch event, that is executed and therefore whole condition evaluates to true in that case and no search is performed. See me debugging this below:

$this->Conn->Query($sql)

This sounds like correct one (search table exists = don't drop it considering that keywords match to previously ones used).