Support from Acquia helps fund testing for Drupal Acquia logo

Comments

farfanfelipe’s picture

Assigned: Unassigned » farfanfelipe
SlayJay’s picture

Assigned: farfanfelipe » SlayJay
SlayJay’s picture

Assigned: SlayJay » Unassigned
SlayJay’s picture

Assigned: Unassigned » SlayJay
SlayJay’s picture

FileSize
1.02 KB

renamed function names.

SlayJay’s picture

Status: Active » Needs review
SlayJay’s picture

FileSize
1.03 KB

forgot to add public.

hoZt’s picture

Status: Needs review » Reviewed & tested by the community

Looks good.

Status: Reviewed & tested by the community » Needs work
Issue tags: -Novice, -VDC

The last submitted patch, 2003246-7449576-2.patch, failed testing.

SlayJay’s picture

Status: Needs work » Needs review
Issue tags: +Novice, +VDC

#7: 2003246-7449576-2.patch queued for re-testing.

oenie’s picture

Status: Needs review » Needs work

Unfortunately we need a reroll, the patch doesn't apply anymore.

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
1.03 KB

re-rolled

Status: Needs review » Needs work

The last submitted patch, views-core-sortOptions-2003246-12.patch, failed testing.

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
1.04 KB

patch 12 no longer applies.

re-rolled

aspilicious’s picture

Status: Needs review » Needs work

Should be protected

=> protected function sortOptions() {

nathangervais’s picture

Assigned: SlayJay » nathangervais
Status: Needs work » Needs review
FileSize
1.04 KB
717 bytes

Here's the old patch with an interdiff to add the protected access modifier.

aspilicious’s picture

Status: Needs review » Needs work

Patch odesn't contain the access modifier

dcam’s picture

It looks like the same patch from #13 was mistakenly re-uploaded.

nathangervais’s picture

Status: Needs work » Needs review
FileSize
717 bytes
1.05 KB

Apologies I had thought i was supposed to supply the original patch with the interdiff. Here's the new patch with the interdiff.

dcam’s picture

It's no problem. The interdiff just shows the change you made between the old patch and the new one so that it's easy for reviewers to see the changes that you've made.

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Perfect!

alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed eb60f6f and pushed to 8.x. Thanks!

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