This Bugzilla instance is deprecated, and most Eclipse projects now use GitHub or Eclipse GitLab. Please see the deprecation plan for details.
Bug 240265 - COSMOS UI should show version and build id information
Summary: COSMOS UI should show version and build id information
Status: CLOSED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Cosmos (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows XP
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Jagmit CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-07-10 01:20 EDT by Srinivas Reddy CLA
Modified: 2012-01-03 13:54 EST (History)
1 user (show)

See Also:


Attachments
Patch for cosmos demo (11.66 KB, application/octet-stream)
2008-07-18 14:45 EDT, Jagmit CLA
sadravid: iplog+
Details
Patch for org.eclipse.cosmos.examples.e2e.dr.views (76.90 KB, patch)
2008-07-24 11:11 EDT, Jagmit CLA
sadravid: iplog+
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Srinivas Reddy CLA 2008-07-10 01:20:48 EDT
It would be nice if we can have "About COSMOS UI" option under COSMOS Web interface with the details of version and build id.

Currently, by looking into COSMOS UI there is no way to find out the build id and version information

I guess, this can be done by providing link over COSMOS logo, which shoud then popup dialog with version,build,copyright information etc.
Comment 1 Sheldon Lee-Loy CLA 2008-07-10 09:35:30 EDT
It was agreed that we would change the alternate text on the COSMOS UI  image.

The build should write a script to modify the following file:

cosmos-demo\webapps\COSMOSUI\pages\cosmos\index.jsp

Note this step should happen after the COSMOSUI is built.

The script should open the index.jsp file and replace the ${buildid} text with the actual build id.

Comment 2 Jagmit CLA 2008-07-18 14:45:05 EDT
Created attachment 107853 [details]
Patch for cosmos demo

Patch for cosmos demo
Comment 3 Jagmit CLA 2008-07-18 14:46:33 EDT
Saurabh:

Can you apply the attached patch
Comment 4 Saurabh Dravid CLA 2008-07-18 15:34:02 EDT
Patch is applied.
Comment 5 Jagmit CLA 2008-07-22 10:17:17 EDT
fixed in the latest cosmos build (COSMOS-1.0.0-200807211620), please verify the build. 
Comment 6 Jagmit CLA 2008-07-22 10:18:25 EDT
fixed in the latest cosmos build (COSMOS-1.0.0-200807211620), please verify the
build.
Comment 7 Srinivas Reddy CLA 2008-07-24 07:10:02 EDT
Hi,

Below fix is available under latest i12 candidate driver, however tooltip doesn't display this information, as it can only display title text data, but not alternate text.

Can we move below fix into title text data instead of alternate text ? so that tooltip can show build information.

Thanks,
Srinivas
Comment 8 Jagmit CLA 2008-07-24 09:54:17 EDT
Can you specify, where this fix should be applied
Comment 9 Srinivas Reddy CLA 2008-07-24 10:03:02 EDT
It's the same file where you have provided the fix, ..\COSMOSUI\pages\cosmos\index.jsp

Here is the code, where I am refering.. (the fix was given under alternate text, it should be under title text, so that tooltip can display this information under UI)
"
...
<td height="35" align="left" bgcolor="#FFFFFF"><img title='Cosmos Data Visualization UI' alt='Cosmos Data Visualization UI COSMOS-1.0.0-200807231648'  src='images/bannerLeft.png'/>
</td>
...
"
Comment 10 Jagmit CLA 2008-07-24 10:21:04 EDT
Sorry I do not undertand, where the fix should go 

do I have the change the index.jsp

If this 
<td height="35" align="left" bgcolor="#FFFFFF"><img title='Cosmos Data
Visualization UI' alt='Cosmos Data Visualization UI COSMOS-1.0.0-200807231648' 
src='images/bannerLeft.png'/>
</td>

the title text 

then were is the alternate text. 




 
Comment 11 Srinivas Reddy CLA 2008-07-24 10:31:14 EDT
(In reply to comment #10)
> Sorry I do not undertand, where the fix should go 
> do I have the change the index.jsp
> If this 
> <td height="35" align="left" bgcolor="#FFFFFF"><img title='Cosmos Data
> Visualization UI' alt='Cosmos Data Visualization UI COSMOS-1.0.0-200807231648' 
> src='images/bannerLeft.png'/>
> </td>
> the title text 
> then were is the alternate text. 

Title text is this => title='Cosmos Data Visualization UI' 
Alternate text is this => alt='Cosmos Data Visualization UI COSMOS-1.0.0-200807231648'

Now, with above fix, COSMOS UI still doen't show build information under tooltip message, as it's displaying title text data.

So, what I am asking is this.. just swap title and alt messages as below
title='Cosmos Data Visualization UI COSMOS-1.0.0-200807231648'
alt='Cosmos Data Visualization UI'

The fix should go into the same file, where you have fixed earlier, that is where you have defined ${buildid}. You just need to move ${buildid} location from 'title' section to 'alt' section. 

Hope it's clear now.
Comment 12 Jagmit CLA 2008-07-24 11:11:49 EDT
Created attachment 108357 [details]
Patch for org.eclipse.cosmos.examples.e2e.dr.views
Comment 13 Jagmit CLA 2008-07-24 11:12:51 EDT
Saurabh, can you apply this patch to the plugin org.eclipse.cosmos.examples.e2e.dr.views
Comment 14 Saurabh Dravid CLA 2008-07-24 11:19:35 EDT
Patch applied.
Comment 15 Jagmit CLA 2008-07-24 11:43:28 EDT
Fixed the issue in the latest cosmos build (COSMOS-1.0.0-200807241123) at 
http://build.eclipse.org/technology/cosmos/downloads/builds.php. 

Please verify the fix. 
Comment 16 Srinivas Reddy CLA 2008-07-25 08:32:03 EDT
This is verified on latest i12 candidate driver #2 - COSMOS-1.0.0-200807241123 and found to be working fine.

Thanks,
Srinivas
Comment 17 Srinivas Reddy CLA 2008-07-31 04:51:41 EDT
QA Review for Manual Tests/JUnits:
- No explicit test cases were defined, however it has tested and verified as part of i12 end2end demo.