Support from Acquia helps fund testing for Drupal Acquia logo

Comments

gdf’s picture

Assigned: Unassigned » gdf
gdf’s picture

Status: Active » Needs review
FileSize
3.55 KB

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

The last submitted patch, drupal-set-count-field-2003304.patch, failed testing.

gdf’s picture

Status: Needs work » Needs review
Issue tags: +Novice, +VDC
gdf’s picture

Assigned: gdf » Unassigned

retest was successful - code is not related, and it was a very, very busy day for testBot

oenie’s picture

Status: Needs review » Reviewed & tested by the community

Looks good to me !

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Needs a re-roll

 curl http://drupal.org/files/drupal-set-count-field-2003304.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3633  100  3633    0     0  10471      0 --:--:-- --:--:-- --:--:-- 21000
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/argument/String.php:157
error: core/modules/views/lib/Drupal/views/Plugin/views/argument/String.php: patch does not apply
jibran’s picture

Status: Needs work » Needs review
FileSize
3.55 KB

re-roll

Status: Needs review » Needs work

The last submitted patch, 2003304-8.patch, failed testing.

SpartyDan’s picture

Status: Needs work » Needs review
FileSize
3.55 KB

re-rolled

dawehner’s picture

Status: Needs review » Reviewed & tested by the community

Cool

alexpott’s picture

Status: Reviewed & tested by the community » Needs work

Needs a re-roll

curl https://drupal.org/files/views-core-setFieldCount-2003304-10.patch | git a
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100  3637  100  3637    0     0   3852      0 --:--:-- --:--:-- --:--:--  4717
error: patch failed: core/modules/views/lib/Drupal/views/Plugin/views/argument/String.php:157
error: core/modules/views/lib/Drupal/views/Plugin/views/argument/String.php: patch does not apply
jibran’s picture

Status: Needs work » Needs review
FileSize
3.57 KB

re-roll

aspilicious’s picture

- * - set_count_field: Reset the count field so we get the right paging.
+ * - setCountField: Reset the count field so we get the right paging.

I'm not sure this change is correct. Think this is an array property in stead of function name. Someone needs to verify this. Setting to needs work because I'm prety sure there is indeed a problem.

aspilicious’s picture

Status: Needs review » Needs work
heddn’s picture

Status: Needs work » Needs review
FileSize
4.24 KB

@aspilicious, I would tend to agree with you in #14, however the precedent seems that other fields have already changed similarly in #2001320: Rename Views method add_groupby() to addGroupBy() & #2002144: Rename Views method add_field() to addField().

Since we need a reroll, one is attached.

aspilicious’s picture

Status: Needs review » Reviewed & tested by the community
alexpott’s picture

Status: Reviewed & tested by the community » Fixed

Committed 00a4259 and pushed to 8.x. Thanks!

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