Launchpad has imported 12 comments from the remote bug at
https://bugs.freedesktop.org/show_bug.cgi?id=55462.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2012-09-30T08:56:59+00:00 Korrawit Pruegsanusak wrote:

Table formatting messed in recent master.

* Open attachment 64325
* Compare it with attachment 64326

Since Sep 24 daily build on Windows XP, and in my master build Sep 29 on Linux.
Details will come later.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/0

------------------------------------------------------------------------
On 2012-10-01T09:28:08+00:00 Korrawit Pruegsanusak wrote:

Created attachment 67913
screenshot: buggy formatting

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/1

------------------------------------------------------------------------
On 2012-10-01T09:29:37+00:00 Korrawit Pruegsanusak wrote:

Created attachment 67914
screenshot: buggy formatting

screenshot again ...

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/2

------------------------------------------------------------------------
On 2012-10-01T09:32:57+00:00 Korrawit Pruegsanusak wrote:

Created attachment 67917
screenshot: buggy formatting

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/3

------------------------------------------------------------------------
On 2012-10-01T09:50:46+00:00 Korrawit Pruegsanusak wrote:

Created attachment 67918
screenshot: buggy formatting

Bah, Bugzilla cut off first 4 bytes from .png image, and first 3 bytes
from .jpg image. Trying again ...

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/4

------------------------------------------------------------------------
On 2012-10-01T09:53:13+00:00 Korrawit Pruegsanusak wrote:

Hello Michael,
I've done git bisect and, unfortunately, your commit introduced this bug.
http://cgit.freedesktop.org/libreoffice/core/commit/?id=6d2e09db4a677068095b0bebd08fbbb96620d60c

Could you please have a look here? Thanks :-)

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/5

------------------------------------------------------------------------
On 2012-10-01T11:11:51+00:00 Stephanv778 wrote:

This problem does not occur when imported as .doc, just when importing
as .docx.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/6

------------------------------------------------------------------------
On 2012-10-04T13:17:02+00:00 Mstahl wrote:

thanks Korrawit for finding that!

i realize now that my fix is totally wrong, because i
misunderstood how covered cells are represented in writer tables:

there is only a dummy TableBox with negative span for
vertically merged cells, but not for horizontally merged cells,
so it is valid for a table to have different number
of TableBoxes in eaach row.

need to think of a different solution for the original
problem that some part of the table was apparently not
covered by a cell...

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/7

------------------------------------------------------------------------
On 2012-10-04T18:01:47+00:00 Libreoffice-bugs wrote:

Michael Stahl committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=4113d9664c60d004474dfc1cffbcd7dc50fa6dc4

rhbz#820283, fdo#55462: sw: better fix for DOCX table import crash:


The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/8

------------------------------------------------------------------------
On 2012-10-05T10:35:25+00:00 Stephanv778 wrote:

Just tested with LO-master version 3.7.0.0.alpha0+ (Build ID: b29ecc9)

The merged horizontal cells problem seems to be solved. Thanks, Michael!

Testing with attachment 64325 shows that not all formatting problems are
solved, however. In the second table not all cell boundary lines are
drawn, content of some of the lower cells is lost and the light blue
background object is placed on top, hiding some of the table content.

Importing the same document as DOC works much better.

I'll leave it up to Korrawit to decide whether he agrees about this bug
being fixed.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/9

------------------------------------------------------------------------
On 2012-10-07T20:42:04+00:00 Libreoffice-bugs wrote:

Michael Stahl committed a patch related to this issue.
It has been pushed to "libreoffice-3-6":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=314f92b0da54085499ea06e7ee53bef4b27181db&g=libreoffice-3-6

rhbz#820283, fdo#55462: sw: better fix for DOCX table import crash:


It will be available in LibreOffice 3.6.3.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/10

------------------------------------------------------------------------
On 2012-10-17T11:08:26+00:00 Korrawit Pruegsanusak wrote:

Sorry for the late reply.

@Michael, Thanks for fixing.

@Stephan, Thanks for testing and noticing another bug with detailed
description. Anyway, it is reported at bug 54660. So, this bug is, IMHO,
fixed.

Thanks again for both of you!

Reply at:
https://bugs.launchpad.net/ubuntu/+source/libreoffice/+bug/1097760/comments/11


** Changed in: df-libreoffice
       Status: Unknown => Fix Released

** Changed in: df-libreoffice
   Importance: Unknown => Medium

-- 
You received this bug notification because you are a member of Ubuntu
Bugs, which is subscribed to Ubuntu.
https://bugs.launchpad.net/bugs/1097760

Title:
  sw: better fix for DOCX table  import crash

To manage notifications about this bug go to:
https://bugs.launchpad.net/df-libreoffice/+bug/1097760/+subscriptions

-- 
ubuntu-bugs mailing list
[email protected]
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to