search

theCrag / website

theCrag.com: Add your voice and help guide the development of the world's largest collaborative rock climbing & bouldering platform
https://www.thecrag.com/
112 stars 8 forks source link

Dead ascents page for Cris #1781

Closed brendanheywood closed 9 years ago

brendanheywood commented 9 years ago

This ascent barfs the server:

http://www.thecrag.com/ascents/at/529383438/by/122728137

Can't reproduce in dev, will re-test after next release.

scd commented 9 years ago

this may actually be the source of those strange out memory messages in our apache error log which we have not dealt with yet. All good things come to those who wait, or is that more for wine makers.

scd commented 9 years ago

Another link that is actually broken in dev

http://dev.thecrag.com/ascents/by/andreh/

This works:

http://dev.thecrag.com/ascents/by/andreh/?pageSize=1&page=5

This does not:

http://dev.thecrag.com/ascents/by/andreh/?pageSize=1&page=6

scd commented 9 years ago

I have just done a hot fix of this. Found an infinite loop in line 19 of crumbtrailPartial template. No stopping condition if there was no TLC in ancestor heirachy.

brendanheywood commented 9 years ago

doh!