Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 440797

Summary: "View on site" page does not load
Product: [ECD] Orion Reporter: Michael Rennie <Michael_Rennie>
Component: ClientAssignee: Michael Rennie <Michael_Rennie>
Status: RESOLVED FIXED QA Contact:
Severity: major    
Priority: P3 CC: ken_walker, mamacdon
Version: 6.0   
Target Milestone: 7.0   
Hardware: PC   
OS: Mac OS X   
Whiteboard:

Description Michael Rennie CLA 2014-07-30 11:39:33 EDT
Steps:

1. select org.eclipse.orion.client/bundles/org.eclipse.orion.client.javascript/web/js-tests/javascript/JsMochaSuite.html
2. popup menu > Open Related > View On Site
3. the resulting sites page is blank

Seems there is an exception while trying to load the page:

"TypeError: undefined is not a function
    at h._generateRow (https://orion.eclipse.org/sites/view.js:1257:233)
    at h._generateChildren (https://orion.eclipse.org/sites/view.js:1256:387)
    at h._generate (https://orion.eclipse.org/sites/view.js:1256:105)
    at https://orion.eclipse.org/sites/view.js:1254:427
    at https://orion.eclipse.org/sites/view.js:1379:239
    at a (https://orion.eclipse.org/sites/view.js:712:456)
    at MutationObserver.f (https://orion.eclipse.org/sites/view.js:712:224)"
Comment 1 Ken Walker CLA 2014-07-30 11:54:06 EDT
At orion.eclipse.org it is loading for me. Just started my site.
Comment 2 Mark Macdonald CLA 2014-07-30 12:18:33 EDT
(In reply to Ken Walker from comment #1)
> At orion.eclipse.org it is loading for me. Just started my site.

It's the "View on Site" page that is broken here, the Sites page linked from the side menu is fine.
Comment 4 Michael Rennie CLA 2014-07-30 23:56:14 EDT
(In reply to Michael Rennie from comment #3)

Looks like it was this commit that broke it:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=b82f95f771629471acfcb8ae45aaa8d16b31c08d

the reason being that the sites and viewOnSIte renders don't have updateExpandVisuals implementations in them (since they have no expansion)

Pushed update to the site(s) renderers to add the stub updateExpandVisuals function and added checks in the tree table to ensure the set renderer has the function before calling it:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=1f4df46091445f02e33b5512cad6612891ed2498