[Xerte-dev] Re: Released several fixes to 3.01

Fay Cross Fay.Cross at nottingham.ac.uk
Thu Sep 10 12:50:57 BST 2015


Thanks Tom. I created a new hotfix branch yesterday with the fix in. I've tested it a fair bit and it's on our installation and seems to be ok. I'll leave it until next week before merging it just in case.

Fay

From: xerte-dev-bounces at lists.nottingham.ac.uk [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Tom Reijnders
Sent: 09 September 2015 15:40
To: For Xerte technical developers
Subject: [Xerte-dev] Re: Released several fixes to 3.01

Yes, that's really the best. If you fix it in devlop, and also separately in master, we could end up with conflicts when trying to merge develop into master again.

If it's really a 1 line fix, you could also consider to fix it in master, and merge into develop.

You generally want to do the fix on a hotfix branch (from master), push it to github, ask others to review, cq. test it, and than merge into master and develop.

Tom


On Wed, 9 Sep 2015 14:16:41 +0000 Fay Cross <Fay.Cross at nottingham.ac.uk<mailto:Fay.Cross at nottingham.ac.uk>> wrote

I'm pretty sure there's a problem in this line breaks hotfix code. From what I can work out toolbox.js goes through every page's attributes and if the XML is from the old editor it calls addLineBreaks function to look at whether the attribute text needs to be adjusted so line breaks aren't lost. This works ok for text stored in an attribute but not where the text is in the CDATA (e.g. text, bullets, advanced image pages). I don't think text in here is being looked at.



I can fix this - should I do it in another hotfix then? And then merge into master like below? It will just be a change to 1 file. I just wanted to check as I've only made changes in develop before.



Thanks





From: xerte-dev-bounces at lists.nottingham.ac.uk<mailto:xerte-dev-bounces at lists.nottingham.ac.uk> [mailto:xerte-dev-bounces at lists.nottingham.ac.uk] On Behalf Of Tom Reijnders
Sent: 24 August 2015 09:32
To: For Xerte technical developers
Subject: [Xerte-dev] Released several fixes to 3.01



L.S.,

Over the weekend, I merged the hotfix branch (linebreaks) and several fixes done on develop into master (3.0) and set a annotated tag v3.01. Effectively this means that vesion.txt now shows 3.0 (v3.01...).

Everything is also merged into develop.

As a reminder, if you know a fix has to end up in the current release, you should:

  1.  create a hotfix branch of master
  2.  implement the fix
  3.  test test test
  4.  merge the hotfix branch into master and into develop

Only do this for fatal bugs.

Regards,

Tom


--

--





Tom Reijnders

TOR Informatica

Chopinlaan 27

5242HM Rosmalen

Tel: 073 5226191

Fax: 073 5226196









This message and any attachment are intended solely for the addressee

and may contain confidential information. If you have received this

message in error, please send it back to me, and immediately delete it.



Please do not use, copy or disclose the information contained in this

message or in any attachment.  Any views or opinions expressed by the

author of this email do not necessarily reflect the views of the

University of Nottingham.



This message has been checked for viruses but the contents of an

attachment may still contain software viruses which could damage your

computer system, you are advised to perform your own checks. Email

communications with the University of Nottingham may be monitored as

permitted by UK legislation.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.nottingham.ac.uk/pipermail/xerte-dev/attachments/20150910/c7f6cede/attachment.html>


More information about the Xerte-dev mailing list