Codecov Report

Merging #1133 into develop will decrease coverage by 0.06%.
The diff coverage is 45.45%.

@@              Coverage Diff              @@
##             develop    #1133      +/-   ##
=============================================
- Coverage      75.83%   75.77%   -0.07%     
- Complexity      2534     2538       +4     
=============================================
  Files            130      130              
  Lines           6221     6230       +9     
=============================================
+ Hits            4718     4721       +3     
- Misses          1503     1509       +6     

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 34cc411...c8ce46a. Read the comment docs.

The problem i mentioned, was related to browser locking when window unload event is trigger.
I solved in this way :

this is an example of the code to reproduce the issue :

        View::addTo($app)
            ->setElement('footer')
            ->addClass('ui basic segment mini')
            ->set('footer');

        $sse = jsSSE::addTo($app, ['showLoader' => false]);
        $sse->set(function () use ($sse) {

            $count = 0;
            ignore_user_abort(true); // i think this must be added to the jsSSE class
            while(!connection_aborted()) { // and this check too, obviously without the while
                sleep(1);
                $sse->send(new jsExpression('console.log('.$count.')'));
                $count++;
            }
        });

        $app->html->template->appendHTML('HEAD', '<script type="application/javascript">' . $sse->jsRender() . '</script>');

This code without that change prevent the browser reload

@ibelar , @abbadon1334 Travis is failing probably not due to this PR, but should be fixed to prevent false positives, can you look at it?

@ibelar , @abbadon1334 Travis is failing probably not due to this PR, but should be fixed to prevent false positives, can you look at it?

restart the build and it worked, sometime travis drink beer on saturday, that's it.

restart the build and it worked, sometime travis drink beer on saturday, that's it.

I know, but that is not a long term solution, there is probably some expected but not synchronized/event-based check.

@mvorisek - @abbadon1334

I have update the pr. Will now check if connection need to be keepAlive.
When keepAlive is set to false, it will behave as before (default)
When set to true, it will keep executing, even after user stop execution

I have also add a function that you can use when user stop execution.
Consider this. If use cancel, it will still run until counter reach a minimum value.

Class Counter
{
    public $count = 0;
}

$counter = new Counter();

$sse = \atk4\ui\jsSSE::addTo($app, ['showLoader' => false, 'keepAlive' => true]);

$sse->onAborted(function($t, $c) {
    if ($c->count > 10) {
        //minimum is reach
        exit();
    }

}, $counter);

$sse->set(function () use ($sse, $counter) {
    $test = true;
    while($test) { // and this check too, obviously without the while
        sleep(1);
        $sse->send(new \atk4\ui\jsExpression('console.log('.$counter->count.')'));
        $counter->count++;
        if ($counter->count > 100) {
            $test = false;
        }
    }
});