Repeater and RepeaterMatrix Pages to Trash

[gingebaker]

Short description of the enhancement

A possibilty that a Repeater (or RepeaterMatrix) Item can be trashed when deleted via the Standard PageEdit procedure (Trash Icon in RepeaterLabel)

Steps that explain the enhancement

1. A new Configuration per RepeaterField (Trash Items or Delete permanently)
2. The Delete Procedure on Page Edit is the same. Eventually show another Trash Icon so that a User can see directly if the Item gets trashed or directly deleted (optional)
3. After a RepeaterItem get´s trashed the RepeaterPage is under the Trash tree like other trashed pages.
4. Ideally there is also a restore process, but at least for me that would not be needed.

Current vs. suggested behavior

The new behaviour would add a config option to RepeaterFields that can also trash RepeaterItems when needed. They would not be always deleted permanently.

Why would the enhancement be useful to users?

This would be helpful when a User accidently deletes a RepeaterItem. Now there is also no Confirm Message when deleting RepeaterItems. Especially in RepeaterMatrix Items there can be a whole lotta Content wich is gone forever.

Addons

I did check FieldtypeRepeater.module and found the delete Statement:
https://github.com/processwire/processwire/blob/d935e9b6995124d76e6fda17fd47ab46dc4d3a3c/wire/modules/Fieldtype/FieldtypeRepeater/FieldtypeRepeater.module#L1225
I also tried trashing in from here in a test setup. The Page would get trashed, but the ListView throws an Error afterwards. I guess The MatrixPages would need some Updates here.

[Toutouwai]

I made a quick test by changing a line in FieldtypeRepeater from...

$this->wire('pages')->delete($p, $saveOptions);

...to...

$this->wire('pages')->trash($p);

...and restoring from the trash seemed to work fine.

I expect that adding this feature is not that simple but hopefully it is doable.