mauritsvanrees
commented
10 years ago Sorry for waiting so long before having a look at this.
I like this, it could be useful.
About the bio: I don't see why richtext would pose a problem. If I undo that part of the changes on this pull request in behavior/membraneuser.py and tests/test_member.py, the tests still pass. If you have seen problems with it, can you tell me how to reproduce those? A test would be good, if possible.
Is there any way that I can see within a Plone Site that it actually works? I am trying to add a template or browser view where I can see member properties. I have a pdb in getPropertiesForUser and it never gets called. It should be easy, but it has been a while since I have written code like that. :-)
Can you add a test that actually uses a new property?
Gagaro
The idea behind this work was to be able to fetch any attribute using member properties. This branch is in a project in development for 4 months now, we didn't encounter any issue.
It also includes french translation and a few fixes.
The only thing which could be an issue is the deletion of "description='bio'" from the properties_map. As stated in the file, it can be RichText and thus break the module.
So I think we should talk about it before merging this pull request. I think a solution would be to rename the bio attribute to description and add description as a default element of the whitelist. It is not the best solution as it would break implementation directly using the bio attribute of the member content though.