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

Bug 348600

Summary: [client][git] Push command doesn't display on git log when repo is in subfolder
Product: [ECD] Orion Reporter: Malgorzata Janczarska <malgorzata.tomczyk>
Component: ClientAssignee: Malgorzata Janczarska <malgorzata.tomczyk>
Status: RESOLVED FIXED QA Contact:
Severity: normal    
Priority: P3 CC: Szymon.Brandys, tomasz.zarna
Version: 0.2Flags: Szymon.Brandys: review+
Target Milestone: 0.2   
Hardware: PC   
OS: Windows XP   
Whiteboard:
Bug Depends on: 349195, 349289    
Bug Blocks:    
Attachments:
Description Flags
git-log.js changes
none
gitCommands.js changes none

Description Malgorzata Janczarska CLA 2011-06-07 11:40:50 EDT 
Yet another problem with repositories in subfolders. This time in UI.
We assumed that Push command on git log should be visible only if we are displaying git log for the top folder of the repository. So far UI assumed that repositories may be located only in workspace top folders and displayed the Push command based on this knowledge. But now when we can have the repository top folder in the subfolder we can't use this assumption any more. We need something more reliable.

Any ideas how can I distinguish between a main folder of the repository and its subfolder?
Comment 1 Tomasz Zarna CLA 2011-06-07 12:02:26 EDT 
(In reply to comment #0)
> Any ideas how can I distinguish between a main folder of the repository and its
> subfolder?

You could compare CloneLocation and your current Location. If they differ it means you're not in a root of a git repository.
Comment 2 Tomasz Zarna CLA 2011-06-07 12:05:14 EDT 
btw, have a look at bug 344746.
Comment 3 Malgorzata Janczarska CLA 2011-06-13 10:46:00 EDT 
Created attachment 197889 [details]
git-log.js changes
Comment 4 Malgorzata Janczarska CLA 2011-06-13 10:46:31 EDT 
Created attachment 197890 [details]
gitCommands.js changes
Comment 5 Malgorzata Janczarska CLA 2011-06-13 10:47:51 EDT 
I added a fix that requires support for deferred on visibleWhen, so I'm waiting for review on bug 349195, then it will have sense to push this changes.
Comment 6 Malgorzata Janczarska CLA 2011-06-15 04:55:13 EDT 
done