spadarian / docblock-python

Atom plugin to insert documentation blocks for python functions
GNU General Public License v2.0
22 stars 9 forks source link

TypeError: Cannot set property 'get_all_docblocks' of undefined #58

Open JonBensi opened 3 months ago

JonBensi commented 3 months ago

Prerequisites

Description

This package docblock-python does not function properly on startup.

Steps to Reproduce

  1. Clone the master branch into the ~/.pulsar/packages directory
  2. Open Pulsar

Expected behavior:

No errors upon startup.

Actual behavior:

Error TypeError: Cannot set property 'get_all_docblocks' of undefined is throw on startup.

Versions

Pulsar: 1.118.2024070703 x64 Electron: 12.2.3 OS: Ubuntu 20.04.6 Thrown From: docblock-python package 0.19.1

Stack Trace

Failed to load the docblock-python package

At Cannot set property 'get_all_docblocks' of undefined

TypeError: Cannot set property 'get_all_docblocks' of undefined
    at Object.<anonymous> (linter-docblock-python.js:6:23)
    at Object.<anonymous> (linter-docblock-python.js:192:2)
    at Module._compile (/app.asar/src/native-compile-cache.js:120:30)
    at /app.asar/src/compile-cache.js:252:23)
    at Module.load (internal/modules/cjs/loader.js:935:32)
    at Module._load (internal/modules/cjs/loader.js:776:14)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12913)
    at Function.o._load (electron/js2c/renderer_init.js:33:379)
    at Module.require (internal/modules/cjs/loader.js:959:19)
    at require (/app.asar/src/native-compile-cache.js:67:27)
    at Object.<anonymous> (docblock-python.js:7:1)
    at Object.<anonymous> (docblock-python.js:983:2)
    at Module._compile (/app.asar/src/native-compile-cache.js:120:30)
    at /app.asar/src/compile-cache.js:252:23)
    at Module.load (internal/modules/cjs/loader.js:935:32)
    at Module._load (internal/modules/cjs/loader.js:776:14)
    at Function.f._load (electron/js2c/asar_bundle.js:5:12913)
    at Function.o._load (electron/js2c/renderer_init.js:33:379)
    at Module.require (internal/modules/cjs/loader.js:959:19)
    at require (/app.asar/src/native-compile-cache.js:67:27)
    at requireModule (/app.asar/src/module-utils.js:3:17)
    at Package.requireMainModule (/app.asar/src/package.js:916:27)
    at /app.asar/src/package.js:141:16
    at Package.measure (/app.asar/src/package.js:82:19)
    at Package.load (/app.asar/src/package.js:127:10)
    at PackageManager.loadAvailablePackage (/app.asar/src/package-manager.js:701:10)
    at /app.asar/src/package-manager.js:609:14
    at Config.transact (/app.asar/src/config.js:892:14)
    at PackageManager.loadPackages (/app.asar/src/package-manager.js:607:17)
    at /app.asar/src/atom-environment.js:973:21

Commands

Non-Core Packages

docblock-python 0.19.1 

Additional Information

I will have a PR to fix this since all that needs to happen is to remove the this keyword from the get_all_docblocks in linter-docblock-python.js

I'm not expert in Javascript, so is this acceptable?