My installation has all the try-out modules installed in /modules/contrib , so the hard-coded path /modules/netnews/nntpserver.inc is incorrect.

Drupal supports loading modules from anywhere, and also supports letting you know where the module actually is - via drupal_get_path()

So I've patched twice:

netnews.module 2005/09/09

Line 842
  require_once(drupal_get_path("module","netnews") .'/nntpserver.inc');

Line 1693
    $fp = fopen(drupal_get_path("module","netnews").'/'.$runtime['name'].'.trace', 'a');

... I think you'll find this good practice ;-)
I was very happy when I learnt about drupal_get_path

.dan.

Comments

janb’s picture

janb commented
Assigned: Unassigned » janb
Status: Needs review » Reviewed & tested by the community

Ah, thanks, since I split the module up in more files now, it becomes even more necessary to do this the right way.