Cannot read property 'includeInto' of undefined

[Asteb612]

tablr version 1.8.3
atom version 1.27.0 with atom-ide
This message is on the open view.

[higorhi]

After updating Atom, Tablr will no longer open Table Editor issuing the same error:
Cannot read property 'includeInto' of undefined

Still shows the preview, though, except after selecting Table Editor mode.

Atom 1.28.0
Tablr 1.8.3

[TakashiAoki]

I am also.
I get the same error message.

[duncdrum]

same on macOS 10.13.5

[brundonsmith]

Same here, Windows 10 Atom 1.28

[rvsandeep]

Same here on Ubuntu 18.04

[syrsly]

Same here, just submitted a duplicate issue, sorry about that.

[kguidonimartins]

Same here on Ubuntu 16.04

[ludovicriffault]

Same here with Atom 1.28.0 x64 on macOS High Sierra

[pjnr1]

Same here;
Atom 1.28.0
macOS High Sierra 10.13.4
tablr 1.8.3

[acrenwelge]

Same here
Atom 1.28.0
Windows 10

[franiis]

Same in:
Atom 1.28.1 x64
tablr 1.8.3
Windows 10

[wataru129]

Same here
atom 1.28.0
macOS High Sierra 10.13.4

[pstaubs]

Issue persists in atom 1.28.2/tablr 1.83 and essentially renders the package usless.

[dsananez]

Same here. Atom 1.28.2 W10. Could someone fix this? ty

[Semihal]

Likewise.

[spettifer-fdb]

Given there have been no commits for at least a year I'd guess the author is no longer supporting this so unless it turns out to be a core problem in Atom (seems unlikely since more people would be shouting about broken stuff), then it's simply a breaking change in Atom that may or may not get fixed in tablr depending on whether or not the author resurrects it or someone submits a PR or forks it. I'd have a crack but I suspect it's well beyond my javascript skills.

[calgacus]

Seems to hit a snag deep in vm.js line 56:
return realRunInContext.call(this, contextifiedSandbox, options);return realRunInContext.call(this, contextifiedSandbox, options);
That return the undefined that gets returned to pool.js around line 66:

const mixin = vm.runInContext(source, context, ${plural}-pool.vm)
mixin.includeInto(this) //-- this is where the exception gets thrown.

All those runInContext APIs are new to me so I doubt I can take this anywhere anytime soon. Shame, I had just found this package and for a few tests it worked great then I hit this bug and which renders it useless, it could have been very useful. I really hope it gets fixed.

[mfripp]

@calgacus, it looks like I was debugging this at the same time as you (wish I'd seen your comment first, although it might have scared me off!). I just posted a pull request that seems to fix this, at least for me. If you want, you can install it via

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

I don't intend to do anything more with this fork, but it may help people use this great package until the main code gets fixed.

[calgacus]

Thanks! It works for me.

[tomasztyzaj]

Thanks @mfripp, worked for me too on MacOS High Sierra 10.13.6 and Atom 1.28.2

[spettifer-fdb]

Yep, W10 Atom 1.28.2 working fine. Thanks for the fix!

[a-paul-taylor]

Fixed it for me (1.28.2 W7) - thanks!

[kguidonimartins]

Thanks, @mfripp! Working fine too on Ubuntu 16.04 and Atom 1.28.1

[piuccio]

Seems to be broken again on Atom 1.29.0

[spettifer-fdb]

@piuccio Are you sure you haven't accidentally updated the package? The fork provided by @mfripp will show as being an older version so Atom's package manager will 'update' it to 1.8.3 if you let it and that will overwrite the fix. Until an official release happens, you will need to ignore all updates to tablr if you want it to keep working. I am running Atom 1.29.0 and it is working fine with the fork.

[piuccio]

oh cool, sorry I missed the #100 (comment) about installing from a fork. Yes it works fine now

[spettifer-fdb]

@piuccio Great!

[clearzale]

@calgacus, it looks like I was debugging this at the same time as you (wish I'd seen your comment first, although it might have scared me off!). I just posted a pull request that seems to fix this, at least for me. If you want, you can install it via

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

I don't intend to do anything more with this fork, but it may help people use this great package until the main code gets fixed.

Amazing, thx!

[TSDAdam]

@skytreetower I had the same issue as you. I got it working by doing:

• Install standard tablr, the non-working one
• replace the copy of lib\mixins\pool.js in your own .atom folder with the one here > https://github.com/mfripp/atom-tablr/blob/master/lib/mixins/pool.js

That worked for me on windows.

[philipsimonhall]

For the record, this also worked for me on Fedora 30 with Atom 1.39.1, just remember to have git installed for installation steps to work, that caught me out temporarily.

[PaoloC68]

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

This worked for me too on a Mac.

[s0q]

@skytreetower I had the same issue as you. I got it working by doing:

* Install standard tablr, the non-working one

* replace the copy of lib\mixins\pool.js in your own .atom folder with the one here > https://github.com/mfripp/atom-tablr/blob/master/lib/mixins/pool.js

That worked for me on windows.

This also worked for me on windows, excellent work!

[thebestpol]

@mfripp Thanks for sharing it works

[eirerocks]

@mfripp - thanks that pull request worked for me - don't know what you did but it works!
it would be good if the original author could have a look and update main for latest Atom

[darkn3rd]

Still happening on mine 1.8.3

[layerssss]

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

As of 13 March, 2020, this is still the solution.

[ulysse42]

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

Same problem, can confirm this still works

[maartenderie]

@abe33 I've you're not going to maintain the project; please find someone who will.

[arvinyorro]

@calgacus, it looks like I was debugging this at the same time as you (wish I'd seen your comment first, although it might have scared me off!). I just posted a pull request that seems to fix this, at least for me. If you want, you can install it via

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

I don't intend to do anything more with this fork, but it may help people use this great package until the main code gets fixed.

After 2 years, the fix still works

To anyone else who do not know where the pool.js is located, its in
C:\Users\{username}\.atom\packages\tablr\lib\mixins

[ujLion]

At: %userprofile%.atom\packages\tablr\lib\mixins
replace pool.js with https://raw.githubusercontent.com/mfripp/atom-tablr/master/lib/mixins/pool.js
And relaunch

[BhaveshDiwan]

At: %userprofile%.atom\packages\tablr\lib\mixins
replace pool.js with https://raw.githubusercontent.com/mfripp/atom-tablr/master/lib/mixins/pool.js
And relaunch

Confirmed to fix the issue in MacOS Catalina 10.15.5

[ApplyCyber]

Work for me on Mac. Thank you!!

[TxJson]

At: %userprofile%.atom\packages\tablr\lib\mixins
replace pool.js with https://raw.githubusercontent.com/mfripp/atom-tablr/master/lib/mixins/pool.js
And relaunch

Can confirm, this still works on Ubuntu 20.04.

[Nazareh]

@calgacus, it looks like I was debugging this at the same time as you (wish I'd seen your comment first, although it might have scared me off!). I just posted a pull request that seems to fix this, at least for me. If you want, you can install it via

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

I don't intend to do anything more with this fork, but it may help people use this great package until the main code gets fixed.

that worked for me

[MattHartzell]

@mfripp This worked great for me! THANK YOU!

[johnpitchko]

#100 worked for me, thank you.

[maxfrischknecht]

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

Still works on February 25, 2021 for Atom 1.54.0, on MacOSx 10.15.7.

[jberthet]

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

Still work on March 29, 2021 for Atom 1.55.0 on macOS BigSur 11.2.3

Is there any way that this version become available in the package manager so people don't get confused ?

[ghost]

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

Still work on March 29, 2021 for Atom 1.55.0 on macOS BigSur 11.2.3

Is there any way that this version become available in the package manager so people don't get confused ?

I can confirm this fixes the issue. I'm also on OSX Big Sur (but on 11.3). I'm on Atom 1.57.0 x64.

[ghost]

It appears @abe33 does in fact still occasionally contribute to private repositories. Hopefully they'll see this message and someone else can take over the tablr package on apm. What they could do is remove tablr from apm and someone who wants to actively work on tablr could take over the package name. The latest work the owner of this repo did for a private repo is some time this month (May 2021). The hope they see the message and give someone else the ability to take over the package name is the reason why I'm writing this request and tagging them. I wouldn't have written this request if say they hadn't done any work in like 6+ months on Github.

[gytisrepecka]

@skytreetower I had the same issue as you. I got it working by doing:

* Install standard tablr, the non-working one

* replace the copy of lib\mixins\pool.js in your own .atom folder with the one here > https://github.com/mfripp/atom-tablr/blob/master/lib/mixins/pool.js

That worked for me on windows.

Thanks, this solution worked 👍

[inform880]

@abe33 Can you give one of us rights to this repo to let us take over development?

[EpicCodeWizard]

Hey everyone! I made a quick apm package with the fixed version. Here it is: https://atom.io/packages/CSV-Editor! Here's the apm command: apm install CSV-Editor (I updated the package name so that it is easier to find).

[claypooj21]

Just in case, @mfripp's version can be installed from the atom package installer GUI by typing "mfripp/atom-tablr" into the package search. It still doesn't show up in the search results, but if you're uncomfortable with a terminal (why? I don't know), that's another option.

Thanks @mfripp and @abe33!

[nitramko]

apm install https://github.com/mfripp/atom-tablr.git

Couldn't clone https://github.com/mfripp/atom-tablr.git

[emendir]

@calgacus, it looks like I was debugging this at the same time as you (wish I'd seen your comment first, although it might have scared me off!). I just posted a pull request that seems to fix this, at least for me. If you want, you can install it via

apm uninstall tablr
apm install https://github.com/mfripp/atom-tablr.git

Thanks a tonne!