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

JSDuck & ExtJS6 - Universal app. #669

Open
fairport opened this issue Aug 16, 2017 · 4 comments
Open

JSDuck & ExtJS6 - Universal app. #669

fairport opened this issue Aug 16, 2017 · 4 comments

Comments

@fairport
Copy link

With ExtJS, in a project using both classic & modern toolkits...
See attached config json file...
It seems that once duck finds the first instance of a class (e.g. in the classic src folders or even the common app source folders) , it ignores any subsequent instance of that class (e.g. in the modern src folders)

Am I correct or am I missing a vital setting in my json file?
jsduck.json.txt
Thanks.

@nene
Copy link
Member

nene commented Aug 16, 2017

What would you expect JSDuck to do when it finds two definitions of the same class?

When JSDuck sees repeated definitions of the same class, it merges them together. This is to support the case where methods of one class have been split to several files. Perhaps this is what happens in your case, and it just seems to you that JSDuck is ignoring the other definition.

To better understand the problem though, try to limit your input to as few files as possible. Then reduce these files to as few lines as possible, reaching to minimal amount of source code that still reproduces the problem. From that minimal amount of code it will be much easier to diagnose what the problem might be.

PS. Your configuration file looks fine to me.

@fairport
Copy link
Author

fairport commented Aug 17, 2017

OK. Thanks.
I thought perhaps I'd missed something important in the JSON file.
For example - with the introduction of the "universal app" ... maybe I need to setup classes differently to comply with jsDuck.

Right now, what I am seeing are quite random effects & I am expecting jsDuck to read my mind!
I think I need to use the words Modern & Classic within my class names. That will fix it.

and .. I am very impressed with jsDuck ... I am amazed how well it builds beautiful, professional looking documentation.

@nene
Copy link
Member

nene commented Aug 17, 2017

Glad it works. Though you should be aware I haven't maintained it for several years, so it has zero knowledge of ExtJS6 and other newer stuff.

@dawesi
Copy link

dawesi commented Sep 3, 2024

I know this is no longer being maintained, but would be great if it could detect the classic or modern directory structure and create two trees one for each. ;-)

Maybe I'll have a look at doing that with a fork. Any tips on best way to do this would be appreciated.

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