Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Installation blocked due to undefined variable in ServerManager #533

Open
Mortimer333 opened this issue Dec 29, 2023 · 1 comment
Open

Comments

@Mortimer333
Copy link

In my fresh installation of Pulsar I tried to install php-ide-serenata but even though my installation check resulted in positive outcome, the actual installation failed.

After fiddling with the output in Console I figured out that the property of ServerManager::versionSpecification is always undefined with results in error (can't bring the exact error but it was [TYPE_ERROR] which required the use of either Buffor, DataView or TypedArray but got undefined instead).

I worked around it by replacing unfortunate property with new Int8Array(8) at line 39 of lib/ServerManager.js which resulted in successful installation. Package now works just fine (even after undoing my changes) which made me wonder if it was actual fault on my side or if there some unexpected bug after moving to Pulsar from Atom (I checked the file history and it wasn't changed in any way that could indicate such issue).

ServerManager.js before fix:

39 |            fs.writeFile(this.getVersionSpecificationFilePath(), this.versionSpecification, (error) => {

After fix:

39 |            fs.writeFile(this.getVersionSpecificationFilePath(), new Int8Array(8), (error) => {
@machitgarha
Copy link
Collaborator

I installed the package on clean Pulsar (1.114.0), and no error occurred. Is this still relevant?

Side note: The original maintainer seems too busy (@Gert-dev). He granted me the access to this repo, but I'm not maintaining this actively (I don't have much time to do so), although I wish I could, neither I'm much familiar with Pulsar or Atom package development.

And... Sorry for the late reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants