the-carlisle-group / Acre-Desktop

A simple Dyalog APL IDE plugin that introduces "projects" and allows you to keep your source code in Unicode text files.
MIT License
11 stars 1 forks source link

Erase confirmation #209

Closed PhilLast closed 4 years ago

PhilLast commented 4 years ago

Kai (@aplteam ) has pointed out the following by email:

I have an ordinary namespace #.APLGUI.Markdown2Help that I want to delete. When I enter

]acre.erase #.APLGUI.Markdown2Help -recursive

acre reports nothing to the session, and content of the namespace is not deleted.

Am I missing something?

In fact Kai was missing the log which contained the request for confirmation that always accompanies an Erase request that exceeds fifty individual items.

My reply to him includes:

This is misleading and needs to be changed somehow.

I suggest losing the confirmation entirely.

aplteam commented 4 years ago

I support this for all the -recursive cases. Are there other places were a confirmation is required? If so the confirmation request should go to the session rather than the log.

PhilLast commented 4 years ago

For Erase confirmation kicks in whenever fifty or more items are to be erased computed after and irrespective of the resolution of namespaces. Currently there are no other commands/functions that require it. But you raise a good point. The messages should be in the session if anywhere.

PhilLast commented 4 years ago

Confirmation is no longer demanded anywhere in acre.

]acre.Erase and ⎕SE.acre.Erase require -recursive or ('recursive'1) for ns contents however many.