Skip to content

Protect the required course symbolic links from deletion in the file manager.#3036

Open
drgrice1 wants to merge 1 commit into
openwebwork:WeBWorK-2.21from
drgrice1:file-manager-protect-links
Open

Protect the required course symbolic links from deletion in the file manager.#3036
drgrice1 wants to merge 1 commit into
openwebwork:WeBWorK-2.21from
drgrice1:file-manager-protect-links

Conversation

@drgrice1

@drgrice1 drgrice1 commented Jul 3, 2026

Copy link
Copy Markdown
Member

This just adds the courseLinks in the course environment to the list of directories (and now links) that are checked when an instructor attempts to delete a directory in the file manager.

This is to address issue #2568.

@drgrice1 drgrice1 force-pushed the file-manager-protect-links branch from 93924e6 to a49c52a Compare July 3, 2026 15:56
@pstaabp

pstaabp commented Jul 6, 2026

Copy link
Copy Markdown
Member

Notice that this just pertains to symbolic links like Library. I have a Fitchburg one that points to the Fitchburg directory in Contrib and it was happy to delete it for me.

@drgrice1

drgrice1 commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Yes, this only pertains to the required links. A local library link is not one of the required links.

@drgrice1

drgrice1 commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Note that if you add $courseLinks{Fitchburg} = [ '/actual/path/to/Fitchburg', "$courseDirs{templates}/Fitchburg"]; to localOverrides.conf, then your Fitchburg link would also be protected by this.

@drgrice1

drgrice1 commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

Note that if you check for upgrades in the admin course, it will tell you Link structure needs to be repaired. for any course that does not have that added link, and if you upgrade the course it will create the link.

@drgrice1 drgrice1 force-pushed the file-manager-protect-links branch 2 times, most recently from f3d1a6b to 50a66fa Compare July 7, 2026 18:54
…manager.

This just adds the `courseLinks` in the course environment to the list
of directories (and now links) that are checked when an instructor
attempts to delete a directory in the file manager.

This is to address issue openwebwork#2568.
@drgrice1 drgrice1 force-pushed the file-manager-protect-links branch from 50a66fa to 116f59e Compare July 7, 2026 21:44
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.

2 participants