joakim-eriksson / nb-eslint

ESLint support for nerbeans
MIT License
14 stars 6 forks source link

Plugin tries to scan and index every mimetype #19

Closed Chris2011 closed 5 years ago

Chris2011 commented 6 years ago

When I open an ini file or I create a SQL query/command/file, I can see, inside the log this output:

WARNING [se.jocke.nb.eslint.task.ESLintTaskScanner]: Callback null!!!!!
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: C:\Users\Chrl\.netbeans\scratches\scratch3.txt@d7c2e16a:3c540ec6 does not lie under MultiFileObject@651e3648[PHP/RuntimeLibraries], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: C:\Users\Chrl\.netbeans\scratches\scratch3.txt@d7c2e16a:3c540ec6 does not lie under AbstractFileObject@73fc51bb[root of C:\Users\Chrl\AppData\Roaming\NetBeans\8.2\jsstubs\corestubs.zip[org.openide.filesystems.JarFileSystem@20cc5040]], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: C:\Users\Chrl\.netbeans\scratches\scratch3.txt@d7c2e16a:3c540ec6 does not lie under AbstractFileObject@6f01121b[root of C:\Users\Chrl\AppData\Roaming\NetBeans\8.2\jsstubs\domstubs.zip[org.openide.filesystems.JarFileSystem@5f866e06]], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: C:\Users\Chrl\.netbeans\scratches\scratch3.txt@d7c2e16a:3c540ec6 does not lie under AbstractFileObject@79b0317b[root of C:\Users\Chrl\AppData\Roaming\NetBeans\8.2\jsstubs\reststubs.zip[org.openide.filesystems.JarFileSystem@d836154]], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: C:\Users\Chrl\.netbeans\scratches\scratch3.txt@d7c2e16a:3c540ec6 does not lie under C:\Program Files\NetBeans 8.2\php\phpstubs\phpruntime@1f3dcb06:6d80e36d, not indexing it
WARNING [se.jocke.nb.eslint.task.ESLintTaskScanner]: Callback null!!!!!
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: MultiFileObject@17925775[Databases/SQLCommands/SQL 9.sql] does not lie under MultiFileObject@651e3648[PHP/RuntimeLibraries], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: MultiFileObject@17925775[Databases/SQLCommands/SQL 9.sql] does not lie under AbstractFileObject@73fc51bb[root of C:\Users\Chrl\AppData\Roaming\NetBeans\8.2\jsstubs\corestubs.zip[org.openide.filesystems.JarFileSystem@20cc5040]], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: MultiFileObject@17925775[Databases/SQLCommands/SQL 9.sql] does not lie under AbstractFileObject@6f01121b[root of C:\Users\Chrl\AppData\Roaming\NetBeans\8.2\jsstubs\domstubs.zip[org.openide.filesystems.JarFileSystem@5f866e06]], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: MultiFileObject@17925775[Databases/SQLCommands/SQL 9.sql] does not lie under AbstractFileObject@79b0317b[root of C:\Users\Chrl\AppData\Roaming\NetBeans\8.2\jsstubs\reststubs.zip[org.openide.filesystems.JarFileSystem@d836154]], not indexing it
WARNING [org.netbeans.modules.parsing.impl.indexing.RepositoryUpdater]: MultiFileObject@17925775[Databases/SQLCommands/SQL 9.sql] does not lie under C:\Program Files\NetBeans 8.2\php\phpstubs\phpruntime@1f3dcb06:6d80e36d, not indexing it
INFO [se.jocke.nb.eslint.annotation.ESLintAnnotationProvider]: Start index file text/x-sql

For ini the same but with text/x-ini

So it seems, that your plugin will try to index or did smth with files which is not JS inside or smth else. Is this right? IMHO I don't think so.

Cheers

Chris

joakim-eriksson commented 6 years ago

No it's not indexing that file it is just checking if the file that changed is a javascript file. But logging should be moved down see.

https://github.com/joakim-eriksson/nb-eslint/blob/master/src/se/jocke/nb/eslint/annotation/ESLintAnnotationProvider.java

Chris2011 commented 6 years ago

Ok, thx for explanation. Yes IMHO the message like Start index, should shown inside of the if, if it is javascript. The warning Callback null!!!!! is not really understandable too, I mean yes there is no callback, but callback for what? I saw the method setScope but didn't dive into it.

Chris2011 commented 5 years ago

PR merged. Will be released soon.