Make sure to set an entry page if none was set before (#10510)

* make sure to set an entry page if none is set yet

* fix possible error

* Site searches will set idaction=0, which is actually a valid entry page url, so we test specifically for false

* Add test case where the visit directly starts with an event

* Add system test where the first actions of a visit are events (without URL set), followed by a pageview later
-> the pageview will be set as the entry page

* Update processed api responses

* UI tests: two new fields in log_visit have been updated

* Processed API responses

1 files changed, 5 insertions, 4 deletions

diff --git a/tests/PHPUnit/Fixtures/TwoVisitsWithCustomEvents.php b/tests/PHPUnit/Fixtures/TwoVisitsWithCustomEvents.php
index cd2a714ab2..fc3f12d979 100644
--- a/tests/PHPUnit/Fixtures/TwoVisitsWithCustomEvents.php
+++ b/tests/PHPUnit/Fixtures/TwoVisitsWithCustomEvents.php
@@ -49,6 +49,11 @@ class TwoVisitsWithCustomEvents extends Fixture
         $uselocal = false;
         $vis = self::getTracker($this->idSite, $this->dateTime, $useDefault = true, $uselocal);
 
+        // $vis will start with a pageview, while $vis2 will directly start with the event
+        $vis->setUrl('http://example.org/webradio');
+        $vis->setGenerationTime(333);
+        self::checkResponse($vis->doTrackPageView('Welcome!'));
+
         $this->trackMusicPlaying($vis);
         $this->trackMusicRatings($vis);
         $this->trackEventWithoutUrl($vis);
@@ -81,10 +86,6 @@ class TwoVisitsWithCustomEvents extends Fixture
 
     protected function trackMusicPlaying(PiwikTracker $vis)
     {
-        $vis->setUrl('http://example.org/webradio');
-        $vis->setGenerationTime(333);
-        self::checkResponse($vis->doTrackPageView('Welcome!'));
-
         $this->moveTimeForward($vis, 1);
         $this->setMusicEventCustomVar($vis);
         self::checkResponse($vis->doTrackEvent('Music', 'play', 'La fiancée de l\'eau'));