There is an extra semicolon

As the title, an extra semicolon appears in the #203 of the field_collection.module file.

Comment	File	Size	Author
#14	field_collection-remove_extra_semicolon-1852474-14.patch	893 bytes	jerenus
#12	field_collection-remove_the_extra_semicolon-1852474-12.patch	534 bytes	jerenus
#10	remove_the_extra_semicolon-1852474-10.patch	534 bytes	jerenus
#7	field_collection-remove_extra_semicolon-1852474-7.patch	893 bytes	jerenus
#3	field_collection-remove_extra_semicolon-1852474-3.patch	888 bytes	jerenus
#1	field_collection-remove_extra_semicolon-1852474.patch	888 bytes	jerenus

Comments

Comment #1

jerenus commented 28 November 2012 at 03:38

Status	File	Size
new	field_collection-remove_extra_semicolon-1852474.patch	888 bytes

Here is my patch.

Log in or register to post comments

Comment #2

28 November 2012 at 03:39

Status:

Needs review

» Needs work

The last submitted patch, field_collection-remove_extra_semicolon-1852474.patch, failed testing.

Log in or register to post comments

Comment #3

jerenus commented 28 November 2012 at 03:53

Status:

Needs work

» Needs review

Status	File	Size
new	field_collection-remove_extra_semicolon-1852474-3.patch	888 bytes

New one.

Log in or register to post comments

Comment #4

28 November 2012 at 03:55

Status:

Needs review

» Needs work

The last submitted patch, field_collection-remove_extra_semicolon-1852474-3.patch, failed testing.

Log in or register to post comments

Comment #5

jerenus commented 28 November 2012 at 03:58

Status:

Needs work

» Needs review

#3: field_collection-remove_extra_semicolon-1852474-3.patch queued for re-testing.

Log in or register to post comments

Comment #6

28 November 2012 at 03:59

Status:

Needs review

» Needs work

The last submitted patch, field_collection-remove_extra_semicolon-1852474-3.patch, failed testing.

Log in or register to post comments

Comment #7

jerenus commented 28 November 2012 at 05:04

Status	File	Size
new	field_collection-remove_extra_semicolon-1852474-7.patch	893 bytes

Sorry for submit a lot of mistakes.

Log in or register to post comments

Comment #8

jerenus commented 28 November 2012 at 05:08

Status:

Needs work

» Needs review

Log in or register to post comments

Comment #9

28 November 2012 at 05:09

Status:

Needs review

» Needs work

The last submitted patch, field_collection-remove_extra_semicolon-1852474-7.patch, failed testing.

Log in or register to post comments

Comment #10

jerenus commented 28 November 2012 at 06:02

Status:

Needs work

» Needs review

Status	File	Size
new	remove_the_extra_semicolon-1852474-10.patch	534 bytes

May be this time!!

Log in or register to post comments

Comment #11

28 November 2012 at 06:13

Status:

Needs review

» Needs work

The last submitted patch, remove_the_extra_semicolon-1852474-10.patch, failed testing.

Log in or register to post comments

Comment #12

jerenus commented 28 November 2012 at 07:06

Version:	7.x-1.0-beta4	» 7.x-1.x-dev
Status:	Needs work	» Needs review

Status	File	Size
new	field_collection-remove_the_extra_semicolon-1852474-12.patch	534 bytes

Log in or register to post comments

kscheirer’s picture

Comment #13

English

Vallejo

commented 29 November 2012 at 07:06

Priority:	Normal	» Minor
Status:	Needs review	» Reviewed & tested by the community

Looks good to me if the tests pass.

Log in or register to post comments

Comment #14

jerenus commented 3 December 2012 at 04:34

Status	File	Size
new	field_collection-remove_extra_semicolon-1852474-14.patch	893 bytes

For testing

Log in or register to post comments

fago’s picture

Comment #15

fago

German

Vienna

commented 25 December 2012 at 22:32

Status:

Reviewed & tested by the community

» Fixed

thanks, committed.

Log in or register to post comments

Comment #16

8 January 2013 at 22:40

Status:

Fixed

» Closed (fixed)

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

Log in or register to post comments