Attached is a minor patch that fixes 2 things:

1. The log message in add/story should be the same as that of page module, which now it is (e.g., in its own fieldset).

2. Remove unncessary function call in page module, maybe a slight performance boost ;-)

After this, these files are identical, which they should be.

CommentFileSizeAuthor
modules_5.patch1.48 KBm3avrck
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to HEAD.

Zen’s picture

I was having a discussion with somebody on why multiple patches should be disallowed, and this kinda proves it. Part of this patch duplicates my own. Separate patches for each file will indicate the number of patches in the queue [via the sequence number] for said file.. etc. etc.

However minor they are, this is a waste of time for all parties concerned esp. Dries_ ..

I'm not sure why the project module does not allow for multiple file attachments with each post. It would greatly simplify things..

Some ground rules would be nice..
Just my 0.02..
-K

Anonymous’s picture

Status: Fixed » Closed (fixed)