Bug #39402
Finisher_Redirect - Alias not working, have to use pid
| Status: | Closed | Start date: | 2012-07-31 | |
|---|---|---|---|---|
| Priority: | Could have | Due date: | ||
| Assignee: | - | % Done: | 0% |
|
| Category: | - | |||
| Target version: | - | |||
| Votes: | 0 |
Description
- Possible Bug in EXT:Formhandler v1.3.0
900.class = Finisher_Redirect
900.config {
# Don't use an alias, it doesn't work.
#redirectPage = foo
- Always use numeric pid
redirectPage = 123
}
Related issues
| related to Formhandler - Bug #40362: There might be a bug in Tx_Formhandler_UtilityFuncs.php::... | Closed | 2012-08-29 |
History
Updated by Uwe Schmelzer 10 months ago
Second try:
Problem
=====================================
Alias doesn't work in Finisher_Redirekt
Workaround:
=====================================
Use pid instead of alias
Tested with:
=====================================
-Typo3 4.5.17
-EXT:Formhandler v1.3.0
-PHP 5.3
Cause
=====================================
Unknown.
I believe, it has worked earlier in Formhandler v1.2.x
Might be related to http://forge.typo3.org/issues/34650
Updated by Tommy Juhnke 10 months ago
Hi Uwe,
I can reproduce this bug in TYPO3 4.7.2, formhandler 1.3.0.
Nevertheless, the browser (Firefox 15.0) performs the redirect (can see the new redirect page ID in the address bar), but the original form is displayed and not the "Thank you" page (as intended). Additionally, the browser displays a message window, claiming that the form's content will be send to "another address".
I guess it depends on the Redirect finisher setting "headerStatusCode". According to the docs it defaults to 307. So I've tried to set it to 302 and now the redirecting and target page display works. Sample setup:
1.class = Finisher_Redirect 1.config.headerStatusCode = 302
Maybe this quick fix works for your purposes too.
Best wishes,
tommy
Updated by Uwe Schmelzer 10 months ago
Hi Thomas,
thx for the quick reply!
Thomas Juhnke wrote:
[...] I guess it depends on the Redirect finisher setting "headerStatusCode". According to the docs it defaults to 307. So I've tried to set it to 302 and now the redirecting and target page display works.
Nope.
Tried 301 and 302, both didn't work with my Typo3 4.5.17 installation.
Outlook:
It could be
a) a problem with Typo3 4.5.17 in general, as it seems to work with your Typo3 4.7.2
b) a problem with my specific formhandler setup
c) a problem with the PHP version. I'm using php5.3.14 here.
d) a problem with the webserver. I'm using Mac OS X Lion with MAMP Pro here, for developement.
Cheers
Uwe
Updated by Reinhard Führicht 10 months ago
- Status changed from New to Needs Feedback
- Priority changed from Won't have this time to Could have
- Target version changed from v1.3 to v1.4
Hi Uwe,
Hi Thomas,
@Thomas: This problem is already fixed in trunk. I will release it as soon as I have time.
@Uwe:
I tried with 4.5.17 and 4.7.2:
redirectPage = 18 -> works fine
redirectPage = asdf -> doesn't work
redirectPage = http://www.google.com/ -> works fine
It doesn't work because the call to the core method "getTypolink_Url" reutrns an empty string in case of "asdf" (the page exists).
I don't think that this has worked in earlier versions fo Formhandler or of TYPO3.
Is there a reason why you would use an alias instead of the page ID?
Updated by Uwe Schmelzer 10 months ago
Hi Reinhard,
Reinhard Führicht wrote:
... I don't think that this has worked in earlier versions fo Formhandler or of TYPO3.
It worked in formhandler 1.2.0 and typo3 4.5.17
But that doesn't matter any more.
The Documentation at
http://www.typo3-formhandler.com/documentation/finisher/finisher-redirect/
clearly states, to use the Page ID or URL.
I'm fine with that.
Is there a reason why you would use an alias instead of the page ID?
a) Readability of typoscript code
b) An alias is easier to remember if I type in an url manually. E.g. to check how my "Thank You"-Page for the order form looks like.
Cheers
Uwe
P.S:
Thanks again to all developers for this great extension.
I just love working with it.
Most everything is where I expect it to be.
Updated by Reinhard Führicht 9 months ago
- Status changed from Needs Feedback to Closed
- Target version deleted (
v1.4)
Thanks!