Aliasing of attributes missing about decoda HOT 6 CLOSED

All of the field names changed, check out FilterAbstract to see what they should be.

from decoda.

Hi Miles,

checked it and think it was map attributes which seems not be working correctly.

For example you've got the following configuration for a url tag.

...
'attributes' => array(
    'target' => '/self|new/i'
 ),
'mapAttributes' => array(
    't' => 'target'
),
...

and the string looks as following:
[url t="new"][/url]
In the old style you could use the alias parameter and it the parser handled it like the target attribute. Now I would need to add the t value as well to the attributes to have this correctly handled. (Which make it obsolete to define it in the mapping)

Reason is that in line 1016 in decoda.php only the source is checked for the key without consideration of the mapping. Do you see it as possible to bring back this behaviour?

from decoda.

I think the problem is that this line should be $finalKey instead of $key. The code is pretty much the same thing as it was before.

https://github.com/milesj/php-decoda/blob/master/Decoda.php#L1016

from decoda.

Tried it but then you get a problem with the url tag and the mapping of default to href.
I thought about it to check in https://github.com/milesj/php-decoda/blob/master/Decoda.php#L1016 both attributes, but then you've got still the problem that the pattern is not correctly loaded.

from decoda.

Ok, check out the latest master commit. This should fix it.

It still uses mapAttributes.

from decoda.

Thanks works fine :)

from decoda.