[drupal-devel] [feature] Forum admin screen updates

walkah drupal-devel at drupal.org
Thu Jan 27 20:32:51 UTC 2005


 Project:      Drupal
 Version:      cvs
 Component:    forum.module
 Category:     feature requests
 Priority:     normal
-Assigned to:  ccourtne
+Assigned to:  walkah
 Reported by:  ccourtne
 Updated by:   walkah
 Status:       patch
 Attachment:   http://drupal.org/files/issues/forumadmin5.patch (22.5 KB)

here's another slight update:

* fixes the updates.inc conflict 
* some documentation / help cleanup
* fixed an inconsistency in 'forum' vs. 'forums' for the vocabulary
module
* check for an existing forum vocabulary in _forum_get_vid, in case
something happens to the variable.

walkah



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

August 29, 2004 - 01:50 : ccourtne

Attachment: http://drupal.org/files/issues/forumadmin.patch (18.5 KB)

Added admin screens to maintain forums instead of forcing the user to
add a taxonomy.  In addition the patch hides the forum taxonomy from
the categories screen since the forum module does not support all
features of taxonomies.

The patch still needs clean up work for comments and the delete
function needs to remove forum topics when you remove a forum, but I
wanted to get it out for review.

This patch requires the latest forum speed up patch found here
http://drupal.org/node/view/10308.

Craig

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

September 8, 2004 - 18:25 : ccourtne

Attachment: http://drupal.org/files/issues/forumadmin2.patch (22.84 KB)

New patch that properly deletes forum topics when the forum is deleted,
has all code comments, and updates the help text.

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

September 8, 2004 - 18:40 : ccourtne

some screen shots of the admin screens
http://homepage.mac.com/ccourtne/drupal/forum_add_forum.png
http://homepage.mac.com/ccourtne/drupal/forum_add_forum.png
http://homepage.mac.com/ccourtne/drupal/forum_add_container.png
http://homepage.mac.com/ccourtne/drupal/forum_config.png

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

September 8, 2004 - 18:42 : ccourtne

One more screenshot.

http://homepage.mac.com/ccourtne/drupal/forum_overview.png

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

September 8, 2004 - 19:21 : ccourtne

updating to patch status

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

September 9, 2004 - 00:17 : Dries

Looks good!

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

September 15, 2004 - 00:08 : ccourtne

Attachment: http://drupal.org/files/issues/forumadmin3.patch (22.77 KB)

Screenshots have a new home.

http://www.dawnsedge.com/ccourtne/drupal/forum_add_forum.png
http://www.dawnsedge.com/ccourtne/drupal/forum_overview.png
http://www.dawnsedge.com/ccourtne/drupal/forum_add_container.png
http://www.dawnsedge.com/ccourtne/drupal/forum_config.png

Also updated the patch for current CVS head.

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

September 20, 2004 - 09:19 : walkah

not only do i think this looks very good, but i also think image.module
folks should maybe take notice :) (hint hint).

oh, and weren't we gonna split taxonomy.module into taxonomy.inc +
taxonomy.module at one point?

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

September 21, 2004 - 13:47 : Dries

I'm going to commit as soon CVS HEAD is opened up for development.  In
the mean time, keep refining this patch so other modules can learn from
it.

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

January 26, 2005 - 20:14 : Boris Mann

Is this going to get committed? Talk about it on CS developers list.

Also: can this separate multiple forums? Can it be extended to support
multiple forums?

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

January 27, 2005 - 11:01 : Dries

I'm afraid it does no longer apply.  It's probably fairly easy to update
though.

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

January 27, 2005 - 14:34 : walkah

Attachment: http://drupal.org/files/issues/forumadmin4.patch (22.04 KB)

updated patch. now applies to HEAD.

-- 
View: http://drupal.org/node/10425
Edit: http://drupal.org/project/comments/add/10425





More information about the drupal-devel mailing list