Project

General

Profile

Actions

Bug #9922

closed
RS RS

Empty server pushes are not properly discarded

Bug #9922: Empty server pushes are not properly discarded

Added by Roel Standaert over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Roel Standaert
Target version:
Start date:
03/10/2022
Due date:
% Done:

100%

Estimated time:

Description

WebRenderer has an isDirty() function that checks whether there is any update at all. If we have no updates then triggerUpdate() won't actually cause any updates to be pushed.

However, this doesn't work properly. It checks if collectedJS1_ is empty, and we always add an opening and closing curly brace.

A simple fix but somewhat ugly fix could be to update the check from:

!collectedJS1_.empty()

to:

!(collectedJS1_.empty() || (collectedJS1_.length() == 2 && collectedJS1_.str() == "{}"))

Related issues 1 (0 open1 closed)

Related to Bug #10019: Follow up issue for issue #9922: Empty server pushes are not properly discardedClosedRoel Standaert03/11/2022

Actions

RS Updated by Roel Standaert over 4 years ago Actions #2

  • Status changed from New to InProgress
  • Assignee set to Roel Standaert

RS Updated by Roel Standaert over 4 years ago Actions #3

  • Status changed from InProgress to Review
  • Assignee deleted (Roel Standaert)

RS Updated by Roel Standaert over 4 years ago Actions #4

  • Precedes Bug #10019: Follow up issue for issue #9922: Empty server pushes are not properly discarded added

RS Updated by Roel Standaert over 4 years ago Actions #5

  • Precedes deleted (Bug #10019: Follow up issue for issue #9922: Empty server pushes are not properly discarded)

RS Updated by Roel Standaert over 4 years ago Actions #6

  • Related to Bug #10019: Follow up issue for issue #9922: Empty server pushes are not properly discarded added

RS Updated by Roel Standaert over 4 years ago Actions #7

  • Status changed from Review to Resolved

RS Updated by Roel Standaert over 4 years ago Actions #8

  • % Done changed from 0 to 100

RS Updated by Roel Standaert over 4 years ago Actions #9

  • Assignee set to Roel Standaert

RS Updated by Roel Standaert over 4 years ago Actions #10

  • Status changed from Resolved to Closed
Actions

Also available in: PDF Atom