Fixing NameID value mapping

[vpalmisano]

Fixes #245

[markstos]

Thanks for the contribution.

The patch needs automated test coverage to be accepted. You could break out attrValueMapper into it's own function and then unit test that.

[huntdm07]

I've just added a similar PR that might fix this.

[markstos]

For reference, the PR @huntdm07 is referring to is: #249

[vpalmisano]

Hi all,
with the latest changes, I added the possibility to define a custom attrValueMapper function. With no option set, the default function is used.

[cjbarth]

Since #427 and #245 are closed, should this be closed too @vpalmisano ?

[cjbarth]

This appears to be closed by other PRs. No response from author. Closing.