Tickets :: Comment on [#10326] even more Ulaform refactoring for H4

* Your Email Address
* Spam protection	Enter the letters below: . ..___._.. ,.___ \|\ \|[__ \| \./ [__ \| \\|\| _\|_ \| [___
Comment	>> - I've seen a few cases where you handle legacy lowercase action >> names. How about adding a migration instead to ucfirst the actions in >> the forms table once and for all? > > When I started refactoring I thought that legacy action handling > must be done only in one or two places and that it will be easier. > But now I'm convinced that migration script probably would have been > a better idea. Will prepare a patch later. > >> - You can simplify redirection by using Horde::url('foo.php')->redirect() > > Does adding redirect() automatically assumes that URL is full and I > don't need second parameter? Or do I still need Horde::url('foo.php', > true)->redirect() ?
Attachment
Watch this ticket