Split up Profile module

Rob Loach - November 12, 2007 - 16:51
Project:Drupal
Version:6.x-dev
Component:profile.module
Category:task
Priority:normal
Assigned:Crell
Status:closed
Description

Although the profile module provides template files, it isn't split up into profile.admin.inc or profile.pages.inc. Is there any reason why it isn't?

#1

Crell - November 13, 2007 - 00:58
Assigned to:Anonymous» Crell
Status:active» patch (code needs review)

Primarily because when going through the list of modules I somehow managed to flag it as having no page callbacks to factor out, which is of course flat wrong. Thanks for the catch. :-) Split patch attached. Cuts out just under half the module, too.

AttachmentSize
profile.patch37.2 KB

#2

catch - November 13, 2007 - 12:18
Status:patch (code needs review)» patch (reviewed & tested by the community)

OK I patched, enabled the module, made some fields, filled them in, visited both user profile and /profile page. All seems fine so RTBC.

#3

Dries - November 13, 2007 - 12:28
Status:patch (reviewed & tested by the community)» fixed

Committed to CVS HEAD. Thanks.

#4

Rob Loach - November 14, 2007 - 07:23

Wow, that was fast.

#5

Anonymous - November 28, 2007 - 07:32
Status:fixed» closed

Automatically closed -- issue fixed for two weeks with no activity.

 
 

Drupal is a registered trademark of Dries Buytaert.