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