Project

General

Profile

Actions

Feature #14964

closed

add form fields or something else via TS

Added by Franz Koch over 18 years ago. Updated almost 16 years ago.

Status:
Closed
Priority:
Should have
Category:
felogin
Target version:
-
Start date:
2005-09-10
Due date:
% Done:

0%

Estimated time:
PHP Version:
Tags:
Complexity:
Sprint Focus:

Description

I used the extension tx_extdbauth from Robert Lemke. Therefor it is necessary to submit a hidden variable during the login-process. It would be nice to add a hook or whatever that makes it possible to add some lines of code simply via TS.
I added it locally in my version of tx_newloginbox, but I don't have CVS, so I can't include it myself.

(issue imported from #M1443)

Actions #1

Updated by Sebastian Roth over 18 years ago

hi, please attach your changes (a patch) to this bug, so that i may review it.

Actions #2

Updated by Franz Koch over 18 years ago

well, I made it quick and dirty by passing through a TS-Variable. But I think it would be of more use if it is somehow parsed through stdwrap or if it would be a CObject so that anything could be included.

my simple change (lines 268 - 277)
---
if (count($onSubmitAr)) {
$onSubmit = implode('; ', $onSubmitAr).'; return true;';
$extraHidden = implode(chr(10), $extraHiddenAr);
}

// Hook to insert other fields or code via TS
// for example to enable the external db-auth from robert lemke (tx_rlmpextdbauth_pi1)
if($this->conf['extraHidden']) {
$extraHidden .= chr(10).$this->conf['extraHidden'];
}
---
well its pretty simple and maybe unsecure? This should only be a suggestion. Maybe something like this would be better:
---
$this->local_cObj = t3lib_div::makeInstance('tslib_cObj'); // Local cObj.
$extraHidden .= $this->local_cObj->COA($this->conf['extraHidden_COA']);
---
but I'm not sure if this is the best way for flexibility. I also thought of some kind of wrapping the login-box, but inside the form-tag, so that anything could be appended or prepended. The best way for this would be the possibility for templating - but that is a different feature request.
Actions #3

Updated by Thorsten Kahler about 17 years ago

This issue should be resolved since the extension is template-ready.

Actions #4

Updated by Steffen Kamper almost 16 years ago

fixed in felogin

Actions

Also available in: Atom PDF