Ticket #75 (closed defect: fixed)

Opened 6 years ago

Last modified 15 months ago

History is not accessible if the page is to big

Reported by: NilsLindenberg Owned by: DotMG
Priority: normal Milestone: 1.3.1
Component: handlers Version: 1.1.6.1
Severity: normal Keywords: history
Cc:

Description (last modified by DarTar) (diff)

If you try to access the history for the SuggestionBox you'll get an server-error (500). Perhaps related to the size of the history? --NilsLindenberg

A rough estimate: there have been between 425 and 450 revisions of that page; the current page **source** size is 92316 bytes; given that the page tends to grow but is sometimes "weeded" when suggestions are implemented or moved to a separate page, the total 'history' **source** size could be something like 12.5MB. Even is it's as "small" as 5MB I wouldn't be surprised to have it break //somehow// (DB access, PHP timeout...).

Maybe we should just give up on 'history' and support only revisions? Where or how something like this breaks is hard to tell, it could be entirely dependent on a specific installation. --JavaWoman

Related tickets

Attachments

optimization_history.patch Download (8.7 KB) - added by vincent.fretin 6 years ago.
optimization of the history handler

Change History

Changed 6 years ago by dartar

  • milestone set to 1.1.6.2

Changed 6 years ago by DarTar

  • description modified (diff)
  • milestone changed from 1.1.6.2 to 1.1.6.3

Changed 6 years ago by vincent.fretin

optimization of the history handler

Changed 6 years ago by vincent.fretin

In the patch, I replaced:

"<ins>".$this->Format(implode("\n", $added))."</ins><br />";

by:

$this->Format('&pound;&pound;'.implode("\n", $added).'&pound;&pound;').'<br />';

It's needed if you want to fix #333

Changed 6 years ago by DotMG

  • owner changed from unassigned to DotMG
  • status changed from new to assigned

Changed 6 years ago by DotMG

Done in 4 step.

[207] : Replacing select * by select note, id, time user -- and adding LIMIT $max to the SQL query

[208] : A code cleanup (in respect to WikkaCodingGuidelinesHowTo)

[210] : Allowing viewing history step by step if page is too big. Adding new config entry pagesize_max to limit the size of the output html.

Todo : Allowing viewing revisions (revisions handler) step by step if page has too many revisions

Todo : Fixing #333 for history.php

Changed 6 years ago by DotMG

  • status changed from assigned to closed
  • resolution set to fixed

[211] : Added (forgotten) LoadOldestRevision() method.

[212] : Allowing viewing revisions (revisions handler) step by step if page has too many revisions

Todo : phpDoc for diff.php and revisions.php

Changed 5 years ago by DarTar

  • description modified (diff)

See #475 for a related issue

Changed 3 years ago by DarTar

  • milestone changed from 1.2 to 1.3

Retargeting to 1.3, this ticket has already been closed in trunk, from which 1.3 will be branched. Consider backporting urgent issues to 1.2.X

Changed 15 months ago by BrianKoontz

  • milestone changed from 1.3 to 1.3.1

Updated milestone to 1.3.1

Note: See TracTickets for help on using tickets.