heavysixer / node-pptx

Generate PPTX files on the server-side with JavaScript.
MIT License
159 stars 44 forks source link

issue #99

Open fazalsandhi opened 1 year ago

fazalsandhi commented 1 year ago

Hi! πŸ‘‹

Firstly, thanks for your work on this project! πŸ™‚

Today I used patch-package to patch nodejs-pptx@1.0.1 for the project I'm working on.

Here is the diff that solved my problem:

diff --git a/node_modules/nodejs-pptx/lib/factories/index.js b/node_modules/nodejs-pptx/lib/factories/index.js
index 9d96aa5..3932925 100644
--- a/node_modules/nodejs-pptx/lib/factories/index.js
+++ b/node_modules/nodejs-pptx/lib/factories/index.js
@@ -39,7 +39,7 @@ class PowerPointFactory {
         await zip.loadAsync(data);

         for (let key in zip.files) {
-            if (zip.files.hasOwnProperty(key)) {
+            if (Object.prototype.hasOwnProperty.call(zip.files, key)) {
                 let ext = key.substr(key.lastIndexOf('.'));

                 if (ext === '.xml' || ext === '.rels') {

This issue body was partially generated by patch-package.