There are some places in the code, which uses views_ajax_error, which actually shouldn't be used anymore,
and especially aren't helpful anymore, as the views 7 UI works different then the one in d6.

CommentFileSizeAuthor
#4 drupal-1933364-4.patch3.85 KBdawehner
#1 drupal-1933364-1.patch3.96 KBdawehner
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Active » Needs review
FileSize
drupal-1933364-1.patch3.96 KB

.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Issue tags: -VDC

#1: drupal-1933364-1.patch queued for re-testing.

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

The last submitted patch, drupal-1933364-1.patch, failed testing.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Needs work » Needs review
FileSize
drupal-1933364-4.patch3.85 KB

Rerolled.

R.Hendel’s picture

R.Hendel CreditAttribution: R.Hendel commented
Status: Needs review » Reviewed & tested by the community

I have applied patch #4 and searched with grep. There was no existance of 'views_ajax_error'.
After this I created a view using ajax. As this works fine, I think the patch is ok.

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented
Status: Reviewed & tested by the community » Fixed

I agree

tstoeckler’s picture

tstoeckler
he/him
Primary language German
Location Essen, Germany
CreditAttribution: tstoeckler commented
Status: Fixed » Reviewed & tested by the community

I might have missed it, but I didn't see this go in?!

dawehner’s picture

dawehner
Primary language German
CreditAttribution: dawehner commented

Ehem right, this function still exists and should be removed.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
CreditAttribution: alexpott commented
Status: Reviewed & tested by the community » Fixed

Committed 53f96ed and pushed to 8.x. Thanks!

views_ajax_error() does not actually exist anymore...

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