In-Portal Issue Tracker - In-Portal CMS
Viewing Issue Advanced Details
387 [In-Portal CMS] Optimization task always 2009-10-13 08:17 2010-01-11 22:05
alex  
alex  
normal  
closed 5.0.1  
fixed  
 
none 5.0.2-B1  
0
0000387: Sitemap optimization
I have pretty small sitemap - 25 sections in total with maximal deep level is 3. There are 149 queries total executed on this page and 4 phrases are loaded by individual queries, however they already should be loaded using cached request (used phrase ids are stored on per template bases in PhrasesCache table).
patch sitemap_optimization.patch (1,900) 2009-10-13 08:25
http://tracker.in-portal.org/file_download.php?file_id=107&type=bug
Issue History
2010-01-11 22:05 Dmitry Note Added: 0001350
2010-01-11 22:05 Dmitry Status resolved => closed
2009-10-25 16:09 alex Note Added: 0000986
2009-10-25 16:09 alex Status reviewed and tested => resolved
2009-10-25 16:09 alex Fixed in Version => 5.0.2-B1
2009-10-25 16:09 alex Resolution open => fixed
2009-10-25 16:09 alex Changeset attached 5.0.x r12793
2009-10-25 16:05 alex Note Added: 0000985
2009-10-25 16:05 alex Target Version Icebox => 5.0.2
2009-10-18 01:58 Dmitry Note Added: 0000911
2009-10-18 01:58 Dmitry Assigned To !COMMUNITY => alex
2009-10-18 01:58 Dmitry Status needs testing => reviewed and tested
2009-10-13 08:25 alex Note Added: 0000891
2009-10-13 08:25 alex Assigned To => !COMMUNITY
2009-10-13 08:25 alex Status active => needs testing
2009-10-13 08:25 alex File Added: sitemap_optimization.patch
2009-10-13 08:18 alex Description Updated bug_revision_view_page.php?rev_id=286#r286
2009-10-13 08:17 alex Target Version => Icebox
2009-10-13 08:17 alex New Issue

Notes
(0000891)
alex   
2009-10-13 08:25   
1. 5 database queries were saved on phrase load, because phrases with no_editing parameter specified during their load (in m_Phrase tag) used different caching key, that wasn't recognized by global caching system.

2. 51 database queries, used to calculate Priority field options for every section list were saved. These calculation as useless on Front-End, because nobody uses them. These only useful in administrative console, because there Priority column is shown with all that options.
(0000911)
Dmitry   
2009-10-18 01:58   
Tested ok.

Just a Note that you are using - if (!$this->Application->IsAdmin()) which will be replaced in the other task in 5.0.2.
(0000985)
alex   
2009-10-25 16:05   
Moving to 5.0.2 release as community input during 5.0.2-B1 version testing.
(0000986)
alex   
2009-10-25 16:09   
Fix committed to 5.0.x branch. Commit Message:

Fixes 0000387: Sitemap optimization
(0001350)
Dmitry   
2010-01-11 22:05   
Closing completed tasks.