- Development - lists.drupal.org

Re: [drupal-devel] Hook_settings() confusion
by mike＠fuckingbrit.com 27 Jan '05

27 Jan '05

>It should be administer >> settings >> modulename, so I fixed the docs too. Yeah, sorry, should have noted that I realised it was the wrong place in the docs and checked the right place. I have links for all /core/ modules that implement it. But spam.module (which implements hook_settings) and my blacklist module (which also implements hook_settings) do not show up in this location. The menu generation code is a bit complex for me to grasp in a 15 minute peek when I should be working ;-) but I've been taking a look. It only appears to call hook_menu for modules, never hook_settings: // Build a sequential list of all menu items. $menu_item_list = module_invoke_all('menu', TRUE); spam.module and my module are both enabled just like blog and blogapi. If I go directly to the URL for a module, admin/settings/spam for example, it works, which would suggest that menu.inc is loading it in somewhere that I couldn't see whilst looking, but not marking it as visible perhaps? I can't really spend any more time looking at this today, better do some "real" work I suppose. If anyone could explain to me why I'm not getting spam.module or mine displaying, but am getting the core modules, it would help! Cheers, Mike -------------------------------------------------------------------- mail2web - Check your email from the web at http://mail2web.com/ .

2 1

[drupal-devel] hook_nodeapi() view op suggestion
by Vladimir Zlatanov 27 Jan '05

27 Jan '05

It might be beneficial to to add 'view_pre' and 'view_post' to the list of possible values for op, and implement it of course. Similar to form_*. The arguments are the same. Some example scenarios: * Adding topical banners before the content. * Adding a taxonomy browser before the content. * Adding relevant pages between the content and the footer. You should be able to do that with the view hook, but it is easy to get in a situation, when different nodeapi modules clash and you end with an unwanted result. Vlado

1 0

Re: [drupal-devel] Hook_settings() confusion
by Chris Cook 27 Jan '05

27 Jan '05

On Thu, 27 Jan 2005 07:51:28 -0000, Michael Jervis <mike(a)fuckingbrit.com> wrote: > Any idea what I've missed? The hook_settings() does not provide links. You need to add a link using hook_menu(). http://drupaldocs.org/api/head/function/hook_menu

1 0

[drupal-devel] theme contribution
by stevew＠etmeli.us 27 Jan '05

27 Jan '05

Good day, I'd like to throw the themes I've developed into the mix on Drupal.org. I've applied for a CVS account, once a couple of weeks ago with some plans, and once again this week after releasing the themes. It is quite possible that I missed an email back, but I was wondering what is the usual lag between applying for CVS access and getting it? Perhaps it is a subtle hint that I should not have CVS, but it didn't work. :) I have always been subtlety-impaired. Steve Wright http://theme.etmeli.us/ -- When I could find voice, I shrieked aloud in agony, "Either this is madness or it is Hell." "It is neither," calmly replied the voice of the Sphere, "it is Knowledge." _Flatland_, by Edwin A. Abbott

2 1

[drupal-devel] Hook_settings() confusion
by Michael Jervis 27 Jan '05

27 Jan '05

Hi, According to the API (http://drupaldocs.org/api/head/function/hook_settings) "This hook provides an administrative interface for controlling various settings for this module. A menu item for the module under "site configuration > modules" will appear in the administrative menu when this hook is implemented." However, in my mt-blacklist module I'm working on, I have defined the function, but there is no link anywhere on the menu for the admin. Nor for spam.module which also defines it. Directly entering the url to the settings page for the module however works fine. Any idea what I've missed? (4.5.2 install, CVS Spam) Mike

1 0

[drupal-devel] [feature] Taxonomy Cloud
by Dan Robinson 27 Jan '05

27 Jan '05

Neil asked me to cross post this here - We (CivicActions) are about to begin work on a "Taxonomy Cloud" or transport mechanism. The goal is not about creating taxonomies but allowing them to be easily shared amongst a network of sites. The goal is to allow folks to define a shared taxonomy and then access content between sites. Please see <dev.civicactions.net/?q=node/114> for my initial write up of the "Taxonomy Cloud". This is something that is a deliverable for CivicGroups and we will begin coding early next week. Please comment at <dev.civicactions.net/?q=forum/34> Thanks, Dan P.S. The original message is posted at <http://civicspacelabs.org/node/view/4981>

1 0

[drupal-devel] Re: [contributions:kbahey] /modules/feedback feedback.module
by Dries Buytaert 27 Jan '05

27 Jan '05

drupal-cvs(a)drupal.org wrote: > Log message: > Added configurable path. For example, it can be 'contact' instead of > the default 'feedback'. > > This is useful, for example, when you replace an existing module with > this one, and you do not want to get 404 errors, and preserve the > same old path. Isn't that a little crufty? It sounds to me as if using path aliases would be better suited for this task. -- Dries Buytaert :: http://www.buytaert.net/

1 0

[drupal-devel] needed documentation for block module for 4.6
by Moshe Weitzman 27 Jan '05

27 Jan '05

I am upgrading recipe.module now like a good citizen. I find these instructions unlikely: "The only required changes to modules implementing hook_block() is to be careful about what is returned. Do not return anything if $op is not 'list' or 'view'". But those are the only $op values according to http://drupaldocs.org/api/head/function/hook_block. Perhaps we need to document 'configure' and other operations? I have not followed these changes very closely.

6 7

Re: [drupal-devel] Module for Review
by Boris Mann 27 Jan '05

27 Jan '05

On Wed, 26 Jan 2005 19:56:21 -0000, Michael Jervis <mike(a)fuckingbrit.com> wrote: > I've written another auth module. > > Contributions/modules/bloggerauth > > Could someone review it prior to me posting a project request. It's > working > here: > http://www.fuckingbrit.com/other_sites/drupal/user/login Works For Me (tm). I logged in with both Blogger and Livejournal accounts. -- Boris Mann http://www.bryght.com

1 0

[drupal-devel] [feature] Forum admin screen updates
by Boris Mann 27 Jan '05

27 Jan '05

Project: Drupal Version: cvs Component: forum.module Category: feature requests Priority: normal Assigned to: ccourtne Reported by: ccourtne Updated by: Boris Mann Status: patch 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? Boris Mann Previous comments: ------------------------------------------------------------------------ August 28, 2004 - 22: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 - 15: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 - 15: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 - 15:42 : ccourtne One more screenshot. http://homepage.mac.com/ccourtne/drupal/forum_overview.png ------------------------------------------------------------------------ September 8, 2004 - 16:21 : ccourtne updating to patch status ------------------------------------------------------------------------ September 8, 2004 - 21:17 : Dries Looks good! ------------------------------------------------------------------------ September 14, 2004 - 21: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 - 06: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 - 10: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. -- View: http://drupal.org/node/10425 Edit: http://drupal.org/project/comments/add/10425

1 0