And here we are splitting up blog.module. I also refactored the menu callbacks to not be Drupal 4.6-era design (dear god...), and removed two notices from the feed callbacks.

CommentFileSizeAuthor
#3 blog_14.patch9.07 KBCrell
blog_13.patch9.44 KBCrell

Comments

dries’s picture

dries commented

Do we really want to put feeds in their own include file? I suggest that for now, we add that to blog.pages.inc.

Crell’s picture

Crell commented

I debated that, but when working on aggregator I decided that feeds are a logical grouping of page callbacks. They're also a small grouping. That means proportionally less code to load on a feed request. I split them off here to parallel what I was doing in aggregator (still being un-broken...).

I guess I can go either way; it just seemed logical that if we were splitting admin pages from user pages that we split feed "pages" off as well.

Crell’s picture

Crell commented
StatusFileSize
new blog_14.patch9.07 KB

Same thing, but just one page file this time.

dries’s picture

dries commented
Status: Needs review » Fixed

Thanks Crell. Committed.

Anonymous’s picture

(not verified) commented
Status: Fixed » Closed (fixed)