Task #7218

Merge revision 10194 into the next release branch

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

Target version:
Start date:
Due date:
% Done:


Estimated time:


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?


#1 Updated by Jing Tao over 3 years ago

I will do this, bryce.

#2 Updated by Jing Tao over 3 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

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?

#3 Updated by Jing Tao over 3 years ago

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

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

Also available in: Atom PDF