Support from Acquia helps fund testing for Drupal Acquia logo

Comments

axyjo’s picture

On another note, happy new year, Drupalers!

axyjo’s picture

FileSize
352 bytes

Sorry. I can't believe I messed up on such a simple patch like this.

Status: Needs review » Needs work

The last submitted patch, copyright2.patch, failed testing.

axyjo’s picture

Status: Needs work » Needs review
FileSize
743 bytes
int’s picture

axyjo’s picture

@int: I was working on a new machine, so I realized that some of the settings weren't right. #4 should apply. It also has an additional date change that yours doesn't have.

int’s picture

@axyjo: ok, it's just to help you, let's see if #4 work for testbot, but it's seems that the testbot are lagged...

int’s picture

Status: Needs review » Reviewed & tested by the community

looks fine.. Patch #4

Jon Nunan’s picture

I don't think you need to, or even should, update the jQuery part of the copyright until we actually include a version of jQuery released in 2010.

Dries’s picture

Status: Reviewed & tested by the community » Fixed

Committed to CVS HEAD.

axyjo’s picture

Version: 7.x-dev » 6.x-dev
Status: Fixed » Patch (to be ported)

Backport to D6?

int’s picture

yes

rdrh555’s picture

Assigned: Unassigned » rdrh555
Status: Patch (to be ported) » Needs review
FileSize
649 bytes

Left jQuery alone. If that's wrong, let me know and I'll redo

jhodgdon’s picture

Status: Needs review » Reviewed & tested by the community

Looks good, applies cleanly. Thanks!

Gábor Hojtsy’s picture

Status: Reviewed & tested by the community » Fixed

Thanks, committed.

Status: Fixed » Closed (fixed)

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