Needs review
Project:
Content Profile
Version:
6.x-1.x-dev
Component:
Miscellaneous
Priority:
Normal
Category:
Feature request
Assigned:
Issue tags:
Reporter:
Created:
7 Aug 2009 at 12:04 UTC
Updated:
27 May 2011 at 07:25 UTC
Jump to comment: Most recent file
Comments
Comment #1
alippai commentedFixed the "namespace", added the dependencies and package to the .info file.
Comment #2
fagohm, for sure we can't ship with that in the base module, as it has a significant impact on performance, but it would make sense to add it as extension module. However for that it would need more polishing - as of now I think it would have problems with some cck fields like user references. Perhaps it could reuse the information about the available cck or node tokens somehow. Also it should do something about preventing token loops (user -> node -> user.. ?)
Comment #3
alippai commentedSo it has support for text and number, single value fields. Added support for flag friends module - content profile tokens for requested and requestee.
Comment #4
alippai commentedComment #5
fagoWhat about my comments from #2?
Comment #6
fagoGot your irc message, I see it'S restricted to numbers and textfields now. So it should be fine.
-> I added in to the module and mentioned it in the README. I fixed some coding style issues and also noted that it didn't use 'user' as typen when doing the replacements. I've no time to test this right now, so I'm attaching the proposed changes. -> Testers needed!
@alippai: I removed your @author comment as with the time also others contribute to the module, but I mentioned you in the README as original author. Is it ok that way?
Comment #7
fagooh and I fixed the module to depend on 'content' too.
Comment #8
Arto commentedThe patch in #6 works fine, at least for our purposes: it is exactly what we needed to get Content Profile fields output as part of RSS feeds using the RDF module. Many thanks, Adam and Wolfgang.
Comment #9
fagoalippai told me on IRC that it's fine that way -> committed, thanks!
Comment #11
Heihachi88 commentedSorry for hijacking this thread again. But what to do if i want content profile title field as token?