Patch to bring aggregator in compliance with ANSI standards.

CommentFileSizeAuthor
#8 ansi-aggregator-0.2.d5.patch1.99 KBFreso
#3 ansi-aggregator-0.2.patch2.12 KBhswong3i
aggregator-admin-ansi-standard.patch1.86 KBAnonymous (not verified)

Comments

hswong3i’s picture

hswong3i commented

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

chx’s picture

chx commented
Status: Needs review » Reviewed & tested by the community

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

hswong3i’s picture

hswong3i commented
StatusFileSize
new ansi-aggregator-0.2.patch2.12 KB

better syntax cleanup.

chx’s picture

chx commented

Whao nice catch! Yes #3 is good also.

Freso’s picture

Freso commented

Patch still applies cleanly.

dries’s picture

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

Committed to CVS HEAD.

Freso’s picture

Freso commented
Version: 7.x-dev » 6.x-dev
Status: Fixed » 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.)

Freso’s picture

Freso commented
StatusFileSize
new ansi-aggregator-0.2.d5.patch1.99 KB

And here's a port for 5.x.

gábor hojtsy’s picture

gábor hojtsy
he/him
Primary language Hungarian
Location Hungary
commented
Version: 6.x-dev » 5.x-dev

Committed to 6.x.

drumm’s picture

drumm
he/him
Location NY, US
commented
Status: Reviewed & tested by the community » Fixed

Committed to 5.x.

Anonymous’s picture

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

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