Early Bird Registration for DrupalCon Portland 2024 is open! Register by 23:59 PST on 31 March 2024, to get $100 off your ticket.
Problem/Motivation
AreaTitleWebTest fails currently with PostgreSQL as database backend.
Proposed resolution
Identify and fix the failing tests.
Remaining tasks
Write patch.
User interface changes
None.
API changes
None.
Comment | File | Size | Author |
---|---|---|---|
#4 | interdiff.txt | 595 bytes | bzrudi71 |
#4 | vdc-postgres-2443691-4.patch | 2.19 KB | bzrudi71 |
Comments
Comment #1
bzrudi71 CreditAttribution: bzrudi71 commentedMoving over existing patch from parent issue. Please see comments #3 - #7. Credits to @xjm please.
Comment #2
dawehnerPuh, this is a hard. Replacing that nice number with another cool string.
Comment #3
daffie CreditAttribution: daffie commentedIt all looks good to me.
I can confirm that the test fails for postgreSQL and with the patch the test passes for postgreSQL.
I have only one problem with the patch:
These lines are now double. If this is necessary, can you explain to me why?
Comment #4
bzrudi71 CreditAttribution: bzrudi71 commentedYes, the lines seem doubled by mistake. Patch attached.
Comment #5
daffie CreditAttribution: daffie commentedLooks good to me.
Comment #6
bzrudi71 CreditAttribution: bzrudi71 commentedThanks for review daffie!
Whoever this commits, credits to @xjm please for the initial patch!
Comment #7
webchickCommitted and pushed to 8.0.x. Thanks!
Comment #10
xjmThe double title assert was to confirm that the title was there, then gone, then back. It was duplicated on purpose. :P
Comment #11
xjmAs it indicates here.
Comment #12
xjm...Oh. But it was not supposed to be duplicated twice at the beginning. Never mind. ;)