Minor cleanup, mainly of some code style things.

CommentFileSizeAuthor
#2 aggregator-cleanup.patch5.34 KBjody lynn
aggregator-cleanup.patch5.33 KBjody lynn

Comments

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Needs work 
+++ modules/aggregator/aggregator.module	5 Jan 2011 23:17:59 -0000
@@ -367,7 +367,7 @@ function aggregator_block_configure($del
+      '#options' => drupal_map_assoc(range(2,20)),

Needs space after comma.

Powered by Dreditor.

jody lynn’s picture

jody lynn
she/her
Primary language English
commented
Version: 8.x-dev » 7.x-dev
Status: Needs work » Needs review
StatusFileSize
new aggregator-cleanup.patch5.34 KB

Added that space.

It's so minor, maybe it can get in for next 7 minor release since 8.x doesn't exist yet.

tim.plunkett’s picture

tim.plunkett
he/him
Primary language English
Location Philadelphia
commented
Status: Needs review » Reviewed & tested by the community

Sure, why not?

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.