Patch aggregator.module to comply with ANSI Standards

boydjd - February 26, 2008 - 08:17
Project:Drupal
Version:5.x-dev
Component:aggregator.module
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

Patch to bring aggregator in compliance with ANSI standards.

AttachmentSize
aggregator-admin-ansi-standard.patch1.86 KB

#1

hswong3i - February 26, 2008 - 09:19

This issue is strict forward and simple, +1. BTW, it is duplicated with http://drupal.org/node/199101.
Tested with MySQL and PostgreSQL, pass.

#2

chx - February 27, 2008 - 22:16
Status:patch (code needs review)» patch (reviewed & tested by the community)

hswong3i, you are calling it duplicated with a highly debated and bloated issue.

#3

hswong3i - February 28, 2008 - 09:36

better syntax cleanup.

AttachmentSize
ansi-aggregator-0.2.patch2.12 KB

#4

chx - February 28, 2008 - 20:35

Whao nice catch! Yes #3 is good also.

#5

Freso - April 23, 2008 - 12:11

Patch still applies cleanly.

#6

Dries - April 23, 2008 - 17:35
Status:patch (reviewed & tested by the community)» fixed

Committed to CVS HEAD.

#7

Freso - April 23, 2008 - 17:57
Version:7.x-dev» 6.x-dev
Status:fixed» patch (reviewed & tested by the community)

Patch applies cleanly to 6.x as well. (I'm assuming it should be back ported as it's a bug.)

#8

Freso - April 23, 2008 - 18:01

And here's a port for 5.x.

AttachmentSize
ansi-aggregator-0.2.d5.patch1.99 KB

#9

Gábor Hojtsy - April 25, 2008 - 21:12
Version:6.x-dev» 5.x-dev

Committed to 6.x.

#10

drumm - April 28, 2008 - 07:41
Status:patch (reviewed & tested by the community)» fixed

Committed to 5.x.

#11

Anonymous (not verified) - May 12, 2008 - 07:44
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.