Unable to paste from spreadsheet

[avalanch]

What editor would you use to apply that?

 

[Steffen]

A text editor of your choice. You have to insert the lines starting with a "+" at the specified position. See https://en.wikipedia.org/wiki/Diff#Unified_format

 

[Steffen]

Better workaround:

--- a/src/XF/Html/Renderer/BbCode.php
+++ b/src/XF/Html/Renderer/BbCode.php
@@ -491,6 +491,12 @@ class BbCode
      */
     public function renderCss(Tag $tag, $stringOutput)
     {
+        // Workaround: https://xenforo.com/community/threads/unable-to-paste-from-spreadsheet.223298/
+        if (preg_match('/^\[(th|td)/i', $stringOutput))
+        {
+            return $stringOutput;
+        }
+
         $css = $tag->attribute('style');
         if ($css)
         {

 

[smozgur]

It looks like this bug still exists, and it’s surprising how quiet this thread has been. Is it being discussed somewhere else?

Thanks for the workaround, @Steffen. Although it removes all cell customizations, it does the job. Better to lose customization instead of garbage entries.
Perhaps the following could be an alternative to using preg_match? (Also, I think only td is affected, but it should be fine to include tr as well.)

if (in_array($tag->tagName(), ['td', 'tr']))

 

[PASS]

We are getting a bunch of posts that are all messed up due to this. @Paul B or someone from XF, can you acknowledge there is an issue. Thx!

 

[avalanch]

Hopefully fixed in 2.4

 

[mjda]

Moved one of my sites to XF 2.3 and already have users complaining about this. The nature of the site is such that there are a LOT of copy/paste from spreadsheets so this is a very unexpected, and unwelcomed regression from XF 2.2.

Here's another instance to look into:

When I copy the table into the editor, it looks fine:



However, when I hit preview, it changes the bottom line...



When I submitted the post, it (obviously) took the preview version of the table, which is . You can see the result, after the thread was posted, here: https://xenforo.com/community/threads/table-copy-paste-test.229188/

 

[briansol]

2 options to get around it for now:
remove style from source. namely, right-aligned cells (numbers) cause issues.

and/or, add the code snippie above : https://xenforo.com/community/threads/unable-to-paste-from-spreadsheet.223298/page-2#post-1702825

 

[Anacleto]

This previously worked before upgrading to 2.3

We have the same problem too, in 2.3 and 2.4.
Give a try here instead:

Valori nutrizionali - Composizione Chimica​

Parte edibile	100%	-
Acqua	42 g	-
Carboidrati disponibili	0	-
Carboidrati complessi	0	-
Zuccheri solubili	0	-
Proteine	35,60 g	-
Grassi (Lipidi)	14,20 g	-

[TR]
[td]Composizione chimica[/td][td]valore per 100g[/td][td]RDA (%)[/td]
[/TR]

 

[Anacleto]

Hi, @Chris D did you ever notice that strange behaviour? thanks.

 

[Mr Lucky]

We have the same problem too, in 2.3 and 2.4.

Lucky you, I haven't yet had the chance to test 2.4

 

[Punt Road End]

I'm also facing this issue... I have pasted in from Excel and Sheets - errors as shown throughout this test thread.

But removing formatting get's closer but still not usable.

tables testing

I'm having similar problems with the Last Man Standing sheet (Excel). I've used a direct screenshot of the sheet in the thread. Posting direct from the sheet, the rows and columns get variously messed up according to what the cells contain.

www.puntroadend.com

 

[shoes22]

It looks like this bug still exists, and it’s surprising how quiet this thread has been. Is it being discussed somewhere else?

Thanks for the workaround, @Steffen. Although it removes all cell customizations, it does the job. Better to lose customization instead of garbage entries.
Perhaps the following could be an alternative to using preg_match? (Also, I think only td is affected, but it should be fine to include tr as well.)

if (in_array($tag->tagName(), ['td', 'tr']))

We had to fix this on our own site. The 'th' tag is also affected and should be added to this list.