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

LOS with other nodes (bug?) #56

Open
krisken opened this issue Jul 23, 2014 · 6 comments
Open

LOS with other nodes (bug?) #56

krisken opened this issue Jul 23, 2014 · 6 comments

Comments

@krisken
Copy link

krisken commented Jul 23, 2014

When i go the page of one of my nodes (index.php?node=xxxx) and i click the button "Line of sight (LOS) with other nodes" a new screen appears. So far, so good.

At the left side, I see the node i've selected. At the right side i have to select a node myself. So i click on "change". Normally a list of all my nodes will appear, but now, i get a new window. Exactly the same window when i click on "Line of sight (LOS) with other nodes".

I suppose this is a bug?

@krisken
Copy link
Author

krisken commented Jul 30, 2014

Any idea yet what is going on?

@foamz
Copy link

foamz commented Sep 11, 2014

Same thing is happening with me on version 1.1-0-a1.
Tried using Firefox and Chrome, both with the same result. I end up copy/pasting the url into a new tab and insert the node numbers like that.

@krisken
Copy link
Author

krisken commented Nov 9, 2014

Still no solution for this problem?

@spinza
Copy link
Contributor

spinza commented Dec 12, 2015

Seeing the same. Will try to fix it.

@jammin84
Copy link
Contributor

yeah, we just ignored that, and added a LOS to all AP's and another for all nodes .

Here is an example.
http://wafn.jolly.so/wind/index.php/nodes?node=18

I can submit code if the community is interested?

@jammin84
Copy link
Contributor

Some further investigation into this matter.

The LOS picker works when you click change from an already working link.
ie go to a node > scroll down to links > click on link > displays plot > click change node. works

These are the points of difference in the urls from the working version mentioned above and the bugged method.

working change
http://wafn.jolly.so/wind/index.php/nodes?page=nodes&subpage=plot_link&a_node=18&b_node=10
opens url
http://wafn.jolly.so/wind/index.php/pickup/nodes?object=form_nodes_plot_link.b_node

not working
http://wafn.jolly.so/wind/index.php/nodes/plot_link?a_node=18
opens url
http://wafn.jolly.so/wind/index.php/nodes/plot_link?object=form_nodes_plot_link.b_node
should be
http://wafn.jolly.so/wind/index.php/pickup/nodes/plot_link?object=form_nodes_plot_link.b_node

test (working minus b_node)
http://wafn.jolly.so/wind/index.php/nodes?page=nodes&subpage=plot_link&a_node=18
opens url (incorrectly)
http://wafn.jolly.so/wind/index.php/nodes?object=form_nodes_plot_link.b_node
should be
http://wafn.jolly.so/wind/index.php/pickup/nodes?object=form_nodes_plot_link.b_node

This should help relative to finding which code to resolve the issue.

jammin84 added a commit to jammin84/wind that referenced this issue Dec 14, 2015
fix url ref.

It was previously only within the context of there being a_node && b_node predefined.
jammin84 added a commit to jammin84/wind that referenced this issue Dec 14, 2015
fix url ref.

It was previously only within the context of there being a_node && b_node predefined.
ysamouhos added a commit that referenced this issue Dec 18, 2015
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

4 participants