PHP Notice on the line 748

Hey guys. This is super simple to fix, what’s the hold up? Every update thus far starts spamming logs again. Here’s some help:

$bShouldBreak = ( $time_now - $time_last > 5 ) ? true : false;  //  insert paragraph break if entries are more than 5 seconds apart
          if( $bShouldBreak ) {
            $time_last_break = $time_now;
          }

          if( $bSentenceEnd && $time_now - $time_last_break > 30 ) {  //  is sentenced has ended previously and it's 30 seconds since last break also insert paragraph break
            $bShouldBreak = true;
            $time_last_break = $time_now;
          }

`$time_last_break` is first initialized inside `if`.

Maybe the simplest thing would be to add `$time_last_break = 0` initialization earlier.

• This reply was modified 7 years, 5 months ago by Eduard.

Code still hasn’t been fixed in 0.9.15.

Just curious – do you guys feel like it’s totally OK to keep spamming your customers production site logs with this?

Compared to the in-depth business blog posts at https://foliovision.com/weblog I find the effort here quite disappointing.