Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 400599 - open commit not working
Summary: open commit not working
Status: RESOLVED FIXED
Alias: None
Product: Orion
Classification: ECD
Component: Git (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: 2.0 RC3   Edit
Assignee: Szymon Brandys CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-02-12 12:04 EST by Susan McCourt CLA
Modified: 2013-02-19 00:12 EST (History)
5 users (show)

See Also:
malgorzata.tomczyk: review+
simon_kaegi: review+


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Susan McCourt CLA 2013-02-12 12:04:21 EST
I tried ctrl+shift+h to open a commit from git repo page and nothing happened on Chrome.  The binding does show up in the key help so I think the command itself is failing.
Comment 1 Susan McCourt CLA 2013-02-15 12:30:50 EST
I investigated this a bit since it's a regression.  The command is never being called because the command is contributed to a scope called "reposPageActions"  (plural) but the only commands ever rendered into the page are from the scope "repoPageAction" (singular).

I'm not sure of the intention here.  Is the presence of both scopes a typo or is this a differentiation between the page mode showing multiple repos vs. one repo?  If the difference is intentional, then the fix is to contribute the command to "repoPageAction" (singular) on line 57 of git-repository.js

I just wasn't sure.  This seems like a safe/simple fix.

Note that open commit still works as expected on the git commit page.
Comment 2 Szymon Brandys CLA 2013-02-18 04:42:43 EST
http://localhost:8080/git/reviewRequest.html#ssh://git.eclipse.org/gitroot/orion/org.eclipse.orion.client.git_56a5a719322c4291142373499783da299298de32 Gosia please review. I understand we have +1 from you Simon on it?
Comment 3 Malgorzata Janczarska CLA 2013-02-18 05:23:57 EST
+1: code looks good and works.
Comment 4 Szymon Brandys CLA 2013-02-18 06:36:11 EST
Fixed.