[drupal-devel] [feature] Themeable profile list patch for profile.module

Uwe Hermann drupal-devel at drupal.org
Mon Sep 19 00:24:08 UTC 2005


Issue status update for 
http://drupal.org/node/20479
Post a follow up: 
http://drupal.org/project/comments/add/20479

 Project:      Drupal
 Version:      cvs
 Component:    profile.module
 Category:     feature requests
 Priority:     normal
 Assigned to:  vfilby
 Reported by:  vfilby
 Updated by:   Uwe Hermann
-Status:       patch (code needs review)
+Status:       patch (code needs work)

Patch doesn't apply any more.




Uwe Hermann



Previous comments:
------------------------------------------------------------------------

Thu, 14 Apr 2005 03:16:04 +0000 : vfilby

Attachment: http://drupal.org/files/issues/profile-themeable-user-list.patch (2.08 KB)

Here is a patch for the profile module that provides a way to theme user
lists.  I simply took the last part of the giant if in profile_browse,
moved it to a new function called theme_profile_list and mde
profile_browse call the new function.




------------------------------------------------------------------------

Thu, 30 Jun 2005 04:52:17 +0000 : moshe weitzman

looka useful ... please provide unified diff format. as per
http://drupal.org/handbook




------------------------------------------------------------------------

Thu, 07 Jul 2005 10:39:32 +0000 : robertDouglass

It doesn't really look suitable for theming, though, due to all the
database logic. To make this really a theme function, you'd have to
prepare the data into some variables that you can get by calling
function A, and make a theme function B that outputs the variables. 


Things that can't be in theme functions include SQL and user access
logic.




------------------------------------------------------------------------

Thu, 04 Aug 2005 03:58:09 +0000 : ejort

Attachment: http://drupal.org/files/issues/profile.module.theme_profile_list.patch (2.17 KB)

Hi,


This is a patch providing the same thing, but without taking the
database code into the theme_profile_list function.  I've had this
running on one of my sites [1] for a while and found it necessary to
output a table listing member contact details (note: there aren't many
profile fields accessible by anonymous users in that example, but you
get the idea).  Unfortunately I never got around to separating it from
my other profile.module modifications until now.


Let me know what you think.


Cheers,
Eric
[1] http://www.renegades.com.au/?q=members/contact-lists/all-men







More information about the drupal-devel mailing list