Support from Acquia helps fund testing for Drupal Acquia logo

Comments

paolomainardi’s picture

FileSize
1.17 KB

Status: Needs review » Needs work

The last submitted patch, check_active-1509436.patch, failed testing.

paolomainardi’s picture

Status: Needs work » Needs review
FileSize
1.18 KB

Sorry this is the correct one.

Status: Needs review » Needs work

The last submitted patch, privatemsg-check_active-1509436-2.patch, failed testing.

paolomainardi’s picture

Status: Needs work » Needs review
FileSize
1.15 KB

Sorry, i forgot to add --relative to git diff.

Status: Needs review » Needs work

The last submitted patch, privatemsg-check_active-1509436-3.patch, failed testing.

paolomainardi’s picture

Trying to use a relative path starting from module root.

Berdir’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, privatemsg-check_active-1509436-7.patch, failed testing.

Berdir’s picture

Status: Needs work » Needs review

The patch needs to be created from the root of the *project*, not the module. Meaning, the path needs to be like a/privatemsg_roles/privatemsg_roles.module.

Berdir’s picture

Status: Needs review » Needs work
adnasa’s picture

Status: Needs work » Needs review
FileSize
1.03 KB

one more time.

Status: Needs review » Needs work

The last submitted patch, privatemsg-check_active-1509436-12.patch, failed testing.

Berdir’s picture

Column 'uid' in field list is ambiguous query: SELECT uid FROM simpletest20230users_roles ur JOIN simpletest20230users u ON ur.uid = u.uid WHERE ur.rid = 4 AND u.status = 1 ORDER BY u.uid ASC LIMIT 0, 100

needs to be u.uid.

adnasa’s picture

Status: Needs work » Needs review
FileSize
1.03 KB

thank you for the feedback,
double checked and this should be fine.

Status: Needs review » Needs work

The last submitted patch, privatemsg-check_if_active-1509436-15.patch, failed testing.

paolomainardi’s picture

Hi Berdir, sorry for delay, i didn't have much time. Thanks adnana for submitting the updated patch.

adnasa’s picture

I've failed though :-(
I don't fully understand the test suite.
will study more before I continue submitting patches.

Berdir’s picture

Status: Needs work » Needs review

Status: Needs review » Needs work

The last submitted patch, privatemsg-check_if_active-1509436-15.patch, failed testing.

ptmkenny’s picture

Status: Needs work » Closed (duplicate)

This was fixed in 7.x and is now a candidate for backport here: https://drupal.org/node/834706