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

Does not intelligently track namespaced/nested classes. #7

Open
jdickey opened this issue Jun 18, 2015 · 2 comments
Open

Does not intelligently track namespaced/nested classes. #7

jdickey opened this issue Jun 18, 2015 · 2 comments

Comments

@jdickey
Copy link

jdickey commented Jun 18, 2015

Given a class ProfileBioHeaderBuilder::Builder that makes use of a class ProfileBioHeaderBuilder::Builder::Button, but uses a relative name for the reference to that class, as

      button = Button.build_native(edit_path) if current_user.name == user_name

both classes are listed in the graphs produced by society, but there is no relationship shown between them (and thus no relationships at all to Button). This catastrophically degrades the usefulness of the graphs.

@CoralineAda
Copy link
Owner

Thanks Jeff, we will look into this soon!

On Jun 18, 2015, at 11:11 AM, Jeff Dickey [email protected] wrote:

Given a class ProfileBioHeaderBuilder::Builder https://github.com/jdickey/new_poc/blob/8e3fc50/app/helpers/users_helper/profile_bio_header_builder/builder.rb that makes use of a class ProfileBioHeaderBuilder::Builder::Button https://github.com/jdickey/new_poc/blob/8e3fc50/app/helpers/users_helper/profile_bio_header_builder/button.rb, but uses a relative name for the reference https://github.com/jdickey/new_poc/blob/8e3fc50/app/helpers/users_helper/profile_bio_header_builder/builder.rb#L17 to that class, as

  button = Button.build_native(edit_path) if current_user.name == user_name

both classes are listed in the graphs produced by society, but there is no relationship shown between them (and thus no relationships at all to Button). This catastrophically degrades the usefulness of the graphs.


Reply to this email directly or view it on GitHub #7.

@wuest
Copy link
Contributor

wuest commented Jun 18, 2015

I'm likely to have cycles to look into this tomorrow/this weekend. Thanks, @jdickey!

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

3 participants