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.<br /><br />If it's really a 1 line fix, you could also consider to fix it in master, and merge into develop. <br /><br />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.<br /><br />Tom<br /><br /><br />On Wed, 9 Sep 2015 14:16:41 +0000 Fay Cross <Fay.Cross@nottingham.ac.uk> wrote
<div style="padding-left: 10px; border-left: solid 2px #0000FF;">
<div class="WordSection1">
<p><span style="font-size: 11pt; font-family: Calibri, 'sans-serif'; color: #1f497d;">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.</span></p>
<p> </p>
<p><span style="font-size: 11pt; font-family: Calibri, 'sans-serif'; color: #1f497d;">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.</span></p>
<p> </p>
<p><span style="font-size: 11pt; font-family: Calibri, 'sans-serif'; color: #1f497d;">Thanks</span></p>
<p> </p>
<p> </p>
<div>
<div style="border: none; border-top: solid #B5C4DF 1pt; padding: 3pt 0cm 0cm 0cm;">
<p><strong><span style="font-size: 10pt; font-family: Tahoma, 'sans-serif';" lang="en-us" xml:lang="en-us">From:</span></strong><span style="font-size: 10pt; font-family: Tahoma, 'sans-serif';" lang="en-us" xml:lang="en-us"> xerte-dev-bounces@lists.nottingham.ac.uk [mailto:xerte-dev-bounces@lists.nottingham.ac.uk] <strong>On Behalf Of </strong>Tom Reijnders<br /><strong>Sent:</strong> 24 August 2015 09:32<br /><strong>To:</strong> For Xerte technical developers<br /><strong>Subject:</strong> [Xerte-dev] Released several fixes to 3.01</span></p>
</div>
</div>
<p> </p>
<p>L.S.,<br /><br /> 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...).<br /><br /> Everything is also merged into develop.<br /><br /> As a reminder, if you know a fix has to end up in the current release, you should:</p>
<ol type="1" start="1">
<li class="MsoNormal">create a hotfix branch of master</li>
<li class="MsoNormal">implement the fix</li>
<li class="MsoNormal">test test test</li>
<li class="MsoNormal">merge the hotfix branch into master and into develop</li>
</ol>
<p>Only do this for fatal bugs.</p>
<p>Regards,<br /><br /> Tom<br /><br /><br /></p>
<pre>-- </pre>
<pre>--</pre>
<pre> </pre>
<p> </p>
<pre>Tom Reijnders</pre>
<pre>TOR Informatica</pre>
<pre>Chopinlaan 27</pre>
<pre>5242HM Rosmalen</pre>
<pre>Tel: 073 5226191</pre>
<pre>Fax: 073 5226196</pre>
<pre> </pre>
<p> </p>
</div>
<pre>

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.
</pre>
</div>