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

Bug 351452

Summary:	[client] save in the file compare view doesn't work
Product:	[ECD] Orion	Reporter:	Piotr Janik <janikpiotrek>
Component:	Client	Assignee:	Project Inbox <e4.orion-inbox>
Status:	CLOSED DUPLICATE	QA Contact:
Severity:	normal
Priority:	P3	CC:	eclipse.felipe, libingw, Szymon.Brandys
Version:	0.2
Target Milestone:	---
Hardware:	PC
OS:	All
Whiteboard:	gsoc2011

Description Piotr Janik

2011-07-07 09:58:47 EDT

Save doesn't work in the compare view (i.e. during conflict resolving).

Uncaught TypeError: Object #<Object> has no method 'getFileMetadata'
exports.EditorCommandFactory.EditorCommandFactory.generateEditorCommands.saveCommand.mCommands.Command.nameeditorCommands.js:48

Fix is being prepared.

Comment 1 Piotr Janik

2011-07-07 10:20:33 EDT

Fix: https://github.com/pjanik/orion.client/tree/bug351452

Comment 2 Felipe Heidrich

2011-07-07 10:51:36 EDT

(In reply to comment #1)
> Fix: https://github.com/pjanik/orion.client/tree/bug351452

that does not look righ to me.
I would expect the fix to be in the compare editor not in the editorCommands.js that is share by all other editors.
Couldn't you change the inputManager used by the compare editor to retun _fileMetadata in its implementation of getFileMetadata() ?
(I'm not familiar with the compare editor code...)

Comment 3 Piotr Janik

2011-07-07 10:58:07 EDT

Well, you're right. The problem is that I'm familiar with the compare editor code too. This is just temporary fix, as I don't know why getFileMetadata() isn't available in the compare editor context in contrast to regular editor. 
I will try to investigate this problem in the next few days.

Comment 4 Felipe Heidrich

2011-07-07 11:00:55 EDT

Thanks Piotr.

Libing, do you know what is going on ?

Comment 5 Piotr Janik

2011-07-18 11:16:19 EDT


*** This bug has been marked as a duplicate of bug 352165 ***