getType(): flatten named parameters before calling init()

[amulet1]

This is a proposed fix to #12:

I made getType() method private as it looks like this is not used anywhere outside of Horde_Form. And this is the only place where the named parameters could be passed (there is also two direct calls to init() in Horde_Form_Type_datetime, but these are internal and they use positional parameters already).

I am using params from value returned by about(). I had to adjust it in a few places (non-critical change) to match original parameters names.

Later we could also add support for defaults by adding them to array returned by about() (and should not this method be static?). It is also possible to implement parameters type checks.

With the migration to V3 this will all be reworked, anyway.

[amulet1]

See also #11 and horde/turba#21.

[amulet1]

@TDannhauer Please merge.

[amulet1]

See also #11 and horde/turba#21.

[TDannhauer]

hi @amulet1 , is your proposal following approach A) or B) from the issue?

I suggest we have a short call - maybe tomorrow - to discuss the approaches , pros and cons to select a road to go.
Sorry, as I am not a native PHP developer, i can not discuss solely based on code...

[amulet1]

I chose approach (A) due to several reasons:

• one simple change in getType(), and it also allows to simplify init() methods
• no need to undo anything
• possibility to add support for default values (and further simplify init())
• possibility to add parameter type checks

[TDannhauer]

hm I see my folder names via Exchange ActiveSync being suffixed with [1] - might this be a consequence of the changes as well @amulet1 ?

edit: visible just in MS outlook - might be a client issue

[amulet1]

hm I see my folder names via Exchange ActiveSync being suffixed with [1] - might this be a consequence of the changes as well @amulet1 ?

edit: visible just in MS outlook - might be a client issue

Changes in the PR should not break anything existing.