Unable to paste from spreadsheet

[Famiboards]

Affected version

2.3

I'm trying to copy and paste a table from Microsoft excel into a post, however the editor is unable to handle it.

I have disabled all addons and the issue persist.

I have attached two screenshots. The first one shows what the editor looks like. At first when you paste the table it looks correct. When you toggle BB code, it turns into the image. That is also the final result when you post.

I also attached a picture of the console log.

This previously worked before upgrading to 2.3

 

[briansol]

i can't reproduce it with my sample workbook.

have you tried another workbook? perhaps theres something funky in the sheet source that's causing the issue.

 

[avalanch]

I'm trying to copy and paste a table from Microsoft excel into a post, however the editor is unable to handle it.

I have disabled all addons and the issue persist.

I have attached two screenshots. The first one shows what the editor looks like. At first when you paste the table it looks correct. When you toggle BB code, it turns into the image. That is also the final result when you post.

I also attached a picture of the console log.

This previously worked before upgrading to 2.3

Attach the microsoft excel file here so that we can test it out.

 

[Deleted member 184953]

Attach the microsoft excel file here so that we can test it out.

Especially if it contains access to your bitcoins...

 

[avalanch]

Especially if it contains access to your bitcoins...

Lol funny post... He wouldn't be trying to paste it then now would he...

 

[Famiboards]

The spreadsheet table is simple, it's just:

a	1
b	2
c	3

It doesn't look like I'm able to attach a .xlsx file here and i'm not sure if I"m allowed to link to a third party file uploader.
I've attached a picture of the sheet itself.

Interestingly, I just noticed it doesn't work here as well when I paste it in. I have checked my other xenforo site which is still on 2.2 and it's working fine.

 

[briansol]

confirmed.

a

b

c

d

[td width="48pt"]
1

[/td]
[td]
2

[/td]
[td]
3

[/td]
[td]
4

[/td]​

 

[briansol]

col B seems to be the issue.

a	I
b	II
c	III
d	IV

[td width="48pt"]
1

[/td]
[td]
2

[/td]
[td]
3

[/td]
[td]
4

[/td]​

 

[briansol]

I noted it only did it on NUMBER cols (as they are right aligned. throwing the RIGHT bbcode in there causes the issue

a	A	I
b	B	II
c	C	III
d	D	IV

all letters is ok so long as you don't align it.


As a stop gap, set the numeric cols to left align in your workbook and paste again

 

[avalanch]

I've had trouble with it messing up tables after the update myself. such as...

Original: https://web.archive.org/web/2021100...ortable-2-infinity-cwcheats-npjh-50332-a.html

2.3's version.... https://www.vgcheat.com/threads/phantasy-star-portable-2-infinity-cwcheats-npjh-50332.181792/

One is from archive.org where vbulletin displayed it properly, the other is after the xenforo 2.3 update. Some tables have similar issues.

It's important to not also that xenforo 2.2 did not have any issues, and this seems to have occurred with xenforo 2.3 update.
Even test threads here suffer from the same issue.

https://xenforo.com/community/threads/table-testing-2-3.223346/#post-1693042

 

[avalanch]

Additionally it threw stuff it couldn't process I guess at the bottom...

 

[⭐ Alex ⭐]

Wonder if it's just a froala bug and reproducible on their own demos https://froala.com/wysiwyg-editor/examples/textarea/

 

[Favori]

Our users are rightfully complaining from this bug. We have a lot of posts which contains table. Any quick fix would be great.

 

[avalanch]

Our users are rightfully complaining from this bug. We have a lot of posts which contains table. Any quick fix would be great.

I have not tried it but this may possibly fix it. Let us know.

E

Post in thread 'Slions - Table'

Jul 19, 2024

This add-on is now supported XF 2.3. You can check the pull request:

jQuery removed and JS recoded by Enes3078 · Pull Request #4 · Slion/XenForoAddOnSlionsTable

jQuery removed and JS recoded.

github.com

XF 2.3 supported add-on is in the attachments.

• Enes3078

• 

 

[Enes3078]

I have not tried it but this may possibly fix it. Let us know.

E

Post in thread 'Slions - Table'

Jul 19, 2024

This add-on is now supported XF 2.3. You can check the pull request:

jQuery removed and JS recoded by Enes3078 · Pull Request #4 · Slion/XenForoAddOnSlionsTable

jQuery removed and JS recoded.

github.com

XF 2.3 supported add-on is in the attachments.

• Enes3078

• 

It should solve the problem. Just give it a try.

 

[Mr Lucky]

Paste from Google spreadsheet looks fine at first

Here is screenshot:




But this is how it renders



[td]
1

[/td][td]
2

[/td][td]
3

[/td]
[td]
4

[/td][td]
5

[/td][td]
6

[/td]

The initial bbcode has a FONT

[FONT=Arial][TABLE width="0px"]
[TR]
[RIGHT][td][RIGHT]1[/RIGHT][/td][td][RIGHT]2[/RIGHT][/td][td][RIGHT]3[/RIGHT][/td][/RIGHT]
[/TR]
[TR]
[RIGHT][td][RIGHT]4[/RIGHT][/td][td][RIGHT]5[/RIGHT][/td][td][RIGHT]6[/RIGHT][/td][/RIGHT]
[/TR]
[/TABLE][/FONT]

But when xenForo to strips out the font it also takes out out the TABLE and TD

 

[Steffen]

Ugh, this bug was unexpected. Copy-pasting tables into XenForo 2.3 is completely broken and our users are complaining.

In my tests, the bug is not caused by the FONT BBCode but by the LEFT and/or RIGHT BBCodes that mess up the table structure.

 

[Enes3078]

Ugh, this bug was unexpected. Copy-pasting tables into XenForo 2.3 is completely broken and our users are complaining.

In my tests, the bug is not caused by the FONT BBCode but by the LEFT and/or RIGHT BBCodes that mess up the table structure.

You should give it a try:

E

Post in thread 'Slions - Table'

Jul 19, 2024

This add-on is now supported XF 2.3. You can check the pull request:

jQuery removed and JS recoded by Enes3078 · Pull Request #4 · Slion/XenForoAddOnSlionsTable

jQuery removed and JS recoded.

github.com

XF 2.3 supported add-on is in the attachments.

• Enes3078

• 

 

[Steffen]

Thanks but I'm not keen on installing an add-on to fix this bug. The add-on has more features than fixing this bug which users would likely use and which would break once we uninstall it again.

 

[Steffen]

A possible quick and dirty workaround:

--- a/src/XF/Html/Renderer/BbCode.php
+++ b/src/XF/Html/Renderer/BbCode.php
@@ -1424,6 +1424,12 @@ class BbCode
                    }
                }
 
+               // Workaround: https://xenforo.com/community/threads/unable-to-paste-from-spreadsheet.223298/
+               if (preg_match('/^\[(th|td)/i', $text))
+               {
+                   return $text;
+               }
+
                return "[$alignmentUpper]{$text}[/$alignmentUpper]";
 
            default: