Project

General

Profile

Actions

Task #7218

closed

Merge revision 10194 into the next release branch

Added by Bryce Mecum over 6 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Target version:
Start date:
10/12/2017
Due date:
% Done:

0%

Estimated time:

Description

I could just Slack this but I thought I'd do an Issue since it's after-hours and I didn't want to send an email. I made a change, revision 10194 back in March and it looks like it never got merged onto a branch for release so the bug it fixed is still affecting Metacat installs. I think the reason it didn't get merged is because I didn't know we cherry-picked commits onto branches to manage release but I know this now!

Jing, could you please merge this sometime so it gets deployed during the next deploy cycle?

Actions #1

Updated by Jing Tao over 6 years ago

I will do this, bryce.

Actions #2

Updated by Jing Tao over 6 years ago

Hi Bryce:
I looked at the version 10194.
valley:metacat tao$ svn log v -r 10194
-----------------------------------------------------------------------

r10194 | mecum | 2017-03-23 10:17:34 -0700 (Thu, 23 Mar 2017) | 1 line
Changed paths:
M /trunk/lib/style/skins/metacatui/eml-2/eml-attribute.xsl

Fix a typo that was preventing dateTime attributes from rendering fully
------------------------------------------------------------------------

You only changed one line in one file trunk/lib/style/skins/metacatui/eml-2/eml-attribute.xsl. It seems you changed:
< <xsl:for-each select="datetime">
---

<xsl:for-each select="dateTime">

I compared the file in the trunk and 2.8 branch and they are same:
svn diff https://code.ecoinformatics.org/code/metacat/trunk/lib/style/skins/metacatui/eml-2/eml-attribute.xsl https://code.ecoinformatics.org/code/metacat/branches/METACAT_2_8_BRANCH/lib/style/skins/metacatui/eml-2/eml-attribute.xsl

So your change was already included in the branch.

I also looked the 2.8.5 tag which is the newest metacat release and the change is there as well. So Metacat 2.8.5 should include the change. Is your metacat the newest version?

Actions #3

Updated by Jing Tao over 6 years ago

  • Status changed from New to Closed
  • Target version set to 2.8.5

Talked with Bryce and close the ticket since the merge already was done in Metacat 2.8.5 release.

Actions

Also available in: Atom PDF