Changing from period (.) to comma (,), where a comma is meant.

CommentFileSizeAuthor
#3 470198-D6.patch687 bytesashmiler
menu.module.patch296 bytesfigaro
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

figaro’s picture

figaro CreditAttribution: figaro commented
Priority: Normal » Minor

This has very low priority, since the typo does not occur in 7.x-dev.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Status: Active » Needs review

The patch is not in the right format (see the docs) and I think 6.x is in a string freeze. But, something exists so it "needs review."

ashmiler’s picture

ashmiler CreditAttribution: ashmiler commented
FileSize
470198-D6.patch687 bytes

Have reworked the patch to be in the right format

Status: Needs review » Needs work

The last submitted patch, menu.module.patch, failed testing.

greggles’s picture

greggles
he/him
Primary language English
Location Denver, Colorado, USA
CreditAttribution: greggles commented
Priority: Minor » Normal
Status: Needs work » Needs review

Looks good to me.

figaro’s picture

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

Looks good to me too.

Gábor Hojtsy’s picture

Gábor Hojtsy
he/him
Primary language Hungarian
Location Hungary
CreditAttribution: Gábor Hojtsy commented
Status: Reviewed & tested by the community » Fixed

Thanks, committed, pushed.

Status: Fixed » Closed (fixed)

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