1
99%
master: 99%

Ran 12 Feb 2019 08:36AM UTC

Files 423

Run time 35s

Badge

Embed ▾

Committed 12 Feb 2019 08:25AM UTC coverage: 99.272%. First build

Job # 2.3.1, PUBLISH_THEME_DOC=true COVERALLS_PARALLEL=true

Build Type

push

travis-ci

Committed by

Commit Message

Ensure entries can be filtered by publication date

Trying to filter entries by publication date in the entry admin index
list caused an error of the form:

    Column 'published_at' in on clause is ambiguous

The combination of the `includes(:published_revision)` in
`scoped_collection` and the `:published_revision_published_at` filter
caused the revisions table to be joined twice, making the unqualified
column `published_at` in the query of the `Revision.published` scope
ambiguous.

Since there is no documented way to obtain the current table alias,
the `published` query has been rewritten to use hash syntax instead of
an SQL fragment.

The comparison of `published_until` can be replaced with a date
range. Since `published_at` is only ever set to the current time, its
value can never be in the future. Checking for its presence is
therefore equivalent to checking whether it lies in the past.

Duplicating the `published_at IS NOT NULL` check, to make precendence
of `AND` and `OR` operators explicit.

REDMINE-16490

Coverage Stats

15683 of 15798 relevant lines covered (99.27%)

27.92 hits per line

tf / pageflow / 2087 / 1
99%
master: 99%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2087.1 (2.3.1, PUBLISH_THEME_DOC=true COVERALLS_PARALLEL=true)

tf / pageflow / 2087 / 1 99% master: 99%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 2087.1 (2.3.1, PUBLISH_THEME_DOC=true COVERALLS_PARALLEL=true)

tf / pageflow / 2087 / 1
99%
master: 99%

README BADGES
x