search

nitcalicut / Paathshaala

Social Content Repository for NITC
www.paathshaala.nitc.ac.in
GNU General Public License v3.0
12 stars 7 forks source link

Content duplication in 3 sections of home page #22

Closed jaseemabid closed 12 years ago

jaseemabid commented 12 years ago

Same content repeated in featured, toprated and popular sections. this gives a feeling that we have only 4 videos in the repo. BAD !

nxvipin commented 12 years ago

Content duplication check is resource intensive. Request to each section will require a minimum of three queries to identify and eliminate duplicates. I suggest we combine all three section into one JSON to minimize the number of queries and to speed up the system.

nxvipin commented 12 years ago

A new branch "combined-storybox-json" is created which combines all the three JSON's and eliminates any duplicate content from all the three sections in the front page. Check json/storybox.json.php to see JSON structure. Front end code needs to be modified to utilize the new JSON structure. Issue assigned to @jaseemabid

jaseemabid commented 12 years ago

Lesser calls to server. This should definitely be faster. I'm a bit busy now, but i will get this done. Will notify before any branch merges.

jaseemabid commented 12 years ago

Done ! Check SHA : a58d21963bf9d49f6aca2f0285c96cfd17e316dc Home page call updated. Only 1 ajax request for the whole page now. Better performance. UI generation code rewritten, I think it improved. Some level of error checks with try ... catch blocks. Merging "combined-storybox-json" to "dev" will lead to quite a lot of conflicts. We can do it today evening and once "combined-storybox-json" merged to dev, we can close this issue.