Do not delete GET parameters about contao-haste HOT 7 CLOSED

@Toflar, what do you think?

from contao-haste.

Honestly, I have no idea. I don't know by heart what happens if a form action is page.html?something=yes and I submit new fields. If it does page.html?something=yes&myformparam=yes automatically, then we need an option to tell HasteForm to kill all existing GET-params on request. If however existing GET-params are killed anyway, we need an option to tell HasteForm to keep existing GET-params (via hidden-inputs).

from contao-haste.

existing GET parameters are killed if you use that URL as action. Therefore, IF I pass any parameters, THEN they should automatically be kept.

from contao-haste.

I prefer

$objForm::preserveGetParams();

So one can set whatever action he wants and if this method is called, they're preserved. If we do it like Andy proposed, they're always passed even if I don't want them to be (which means I have to do preg_replace() on my action to remove them :-))

from contao-haste.

Gents, can you have a quick look at this fedbc68? I have moved parsing the "action" parameter to the __get() method, which allows now to e.g. set the action URL before setting the form method. Also, now with GET method the URL paraams are automatically passed to the hidden fields. They can be still altered by getting and setting the $this->hiddenFields.

from contao-haste.

Looks good to me!

from contao-haste.

I have no idea what you did, but I'm sure it's good :P

from contao-haste.