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

Add ability to delete whole paths when deleting a device #1234

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

dorion3d
Copy link

@dorion3d dorion3d commented Jul 8, 2020

This is a suggestion which would add the possibility to select which related connections past the selected device you would like to remove at the same time as deleting that selected device.
In other words, if you need to remove all the connections between two devices you can do it in this screen instead of having to go to each device it interacts with.

The new popup would replace the confirmation to delete the device.

This version makes rear connections not removable but this can be changed.
Types of behaviours it can have:
Checked
Unchecked
Force Check
Force Unchecked

Look at the code in the GetPotentialPathToDelete function in devices.php for more info.

For example, I want to delete Switch 1 which has this one connection in port 2.
switchpath

When I click Delete this would pop up.
switchdel

The first connection is interacting with the device I'm trying to delete so it would have to be deleted. (Force Checked)
The two links with the (Rear) tag are back links between patch panels which are to be kept (Force Unchecked) but that is configurable as mentioned above.
The two unchecked are front connections between two patch panels which in this case, are automatically being checked to be remove (Checked) but can be unchecked if wanted(Also configurable).

If I were to click "Delete device and selected paths"
the device would be deleted and only the rear connections between the patch panels would be kept.
It also makes a excel sheet containing the paths you deleted so you don't forget which paths you removed if you are to be removing the physical paths later.
excelpaths

If you have any questions feel free to ask me!

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

Successfully merging this pull request may close these issues.

1 participant