grahamc/hydra - Change 2T3XCADO3TVJM2TONBE46KC2XMOVO5IPYN6CRQG5HHFTDDPJFH5QC

Build.pm: Don't get isChannel using buildproducts.

We now have a column for that, so no need for counting rows which was a bit inefficient anyway, because we only would have needed the first row in the result.

Signed-off-by: aszlig <aszlig@redmoonstudios.org>

Created by aszlig on April 21, 2015

2T3XCADO3TVJM2TONBE46KC2XMOVO5IPYN6CRQG5HHFTDDPJFH5QC

Dependencies

In channels

main

upstream

Change contents

Deletion in src/lib/Hydra/Controller/Build.pm at line 111 [4.1]

B:BD[3.164] → [2.0:119]


    $c->stash->{isChannel} = $c->stash->{build}->buildproducts->search(
        {type => "channel"}
    )->count > 0;

Replacement in src/root/build.tt at line 191 [6.7350]

B:BD[5.3656] → [2.120:245]

            [% IF (!isAggregate || !isChannel) && build.finished; actualBuild = build.iscachedbuild ? cachedBuild : build %]

[5.3656]

[7.535]

            [% IF (!isAggregate || !build.isChannel) && build.finished; actualBuild = build.iscachedbuild ? cachedBuild : build %]

Replacement in src/root/build.tt at line 203 [6.7350]

B:BD[8.59] → [2.246:321]

            [% IF (!isAggregate || !isChannel) && buildLogExists(build) %]

[8.59]

[9.6492]

            [% IF (!isAggregate || !build.isChannel) && buildLogExists(build) %]

Replacement in src/root/build.tt at line 218 [6.7350]

B:BD[10.843] → [2.322:386]

    [% IF isChannel || (build.buildproducts && !isAggregate) %]

[10.843]

[11.1105]

    [% IF build.isChannel || (build.buildproducts && !isAggregate) %]