Problem/Motivation


testMissingBundleFieldRender
fail: [Other] Line 534 of core/modules/views/src/Tests/Handler/FieldFieldTest.php:
Value '' is equal to value Drupal\Core\Render\SafeString::__set_state(array(
   'string' => '0',
)).

https://www.drupal.org/pift-ci-job/25521

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

CommentFileSizeAuthor
#2 2561135.2.patch1.42 KBalexpott

Comments

catch created an issue. See original summary.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
StatusFileSize
new 2561135.2.patch1.42 KB

Tested on postgres, sqlite, and mysql. VIews really should add a default and mandatory sort order - would save us a lot of postgres issues.

alexpott’s picture

alexpott
he/they
Primary language English
Location 🇪🇺🌍
commented
Status: Active » Needs review
dawehner’s picture

dawehner
Primary language German
commented

Tested on postgres, sqlite, and mysql. VIews really should add a default and mandatory sort order - would save us a lot of postgres issues.

Well, it adds one if you use the UI, it just doesn't add one when you hack your own config file together.
When we would add one, I kinda fear that this would feel kinda hard to debug if people don't see it in the UI.

dawehner’s picture

dawehner
Primary language German
commented
Status: Needs review » Reviewed & tested by the community

.

catch’s picture

catch
he/him
Primary language English
commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 8.0.x, thanks!

  • catch committed 36788f8 on 8.0.x 
    Issue #2561135 by alexpott: Views.Drupal\views\Tests\Handler\...

Status: Fixed » Closed (fixed)

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