force run onSiteChange first time

[flamisz]

refs
matomo-org/matomo#16787
matomo-org/matomo#17367

[flamisz]

@tsteur, @diosmosis already merged this PR.
This is connected to that one, because the tag manager containers are not loading onto the page without this. Can you please you or @sgiehl check it?

[diosmosis]

I'm not super familiar w/ the tag manager, but the code looks good to me

[sgiehl]

@flamisz could you maybe describe a bit clearer what "bug" this should fix?

[flamisz]

@flamisz could you maybe describe a bit clearer what "bug" this should fix?

@sgiehl sure.
For the PR I needed this change, I included the tagmanager component. The same, what we use on the admin/tracking code page.
It works there, but not working here, because it's not calling that onSiteChange when the page loads. I console.log-d the val and oldVal in the $watch method, and this is how it looks like in the admin page and on the "no data yet page":

// admin page
undefined undefined
8 undefined

// no data yet page
8 8 

I'm not sure where that undefined is coming from but looks like it almost just working accidentally. So I added this "firstTime" check.
Now that I'm thinking through, maybe a better way, like run that when the component loads, but this is working too.

[sgiehl]

Now I was able to reproduce and test. Seems to work as expected now.