[ 
http://jira.codehaus.org/browse/CONTINUUM-2262?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Wendy Smoak updated CONTINUUM-2262:
-----------------------------------

    Attachment: CONTINUUM-2262-wsmoak-20090605-1.patch

Attaching work in progress.  I've added the field to the model and put it on 
both of the pages, but haven't managed to get the field populated yet.

Notes:

projectGroupSummary.jsp
 -> contains projectSummaryComponent.jsp 
             - has Last Build Date column
             - needs date for the scm result as well

             What does cell="date" do in <ec:column> ?

scmResult.jsp should also show the scm result date

ScmResult.java has no date field and is generated code.

Need to add a date field to continuum-model continuum.xml 
 - based on the other date fields, it needs to be a long
 - name of field can't be 'date' because it's reserved by some DBMS

Now that ScmResult has a date field (named scmResultDate), search for usages of 
ScmResult class

Found updateWorkingDirectoryFromScmContinuumAction.java and added
result.setScmResultDate( new Date().getTime() );
 - needs a unit test

To test this in the webapp, there is a sandbox project with a bad scm url:
http://svn.apache.org/repos/asf/continuum/sandbox/examples/bad-scm/pom.xml
add it and force a build to see the scm error.

> Display the date on the scm result
> ----------------------------------
>
>                 Key: CONTINUUM-2262
>                 URL: http://jira.codehaus.org/browse/CONTINUUM-2262
>             Project: Continuum
>          Issue Type: Improvement
>    Affects Versions: 1.3.3
>         Environment: 1.3.3, 1.3.4-SNAPSHOT r781487 on vmbuild
>            Reporter: Wendy Smoak
>            Priority: Minor
>         Attachments: CONTINUUM-2262-wsmoak-20090605-1.patch
>
>
> Currently when viewing an scm result, there is no way to tell when an error 
> happened.
> The date should be displayed both on the Project Group Summary (similar to 
> the Last Build Date column) and on the SCM result detail page.
> See the screen shots attached to [the otherwise unrelated] CONTINUUM-2261.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to