Refactoring for Manage features #305
Conversation
Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
Signed-off-by:Thorsten Eurich <thorsten@eurich.de>
|
Thank you! Yes, it makes sense to see if we can take all these pause() functions and define the operations to do, with a pause/continue mechanism. It may help us clean up and/or 'standardize' a bit the implementation in other scripts too? (i.e. upgrade, importer) |
LOL .. I was actually going to ask you to do this one next so I did not mess it up for you 😃 Glad its done, now I can get back to scratching my head on those profile fields |
|
Hahaha, seems to be almost perfect timing. 😺 Good luck with the profile fields The code I've seen so far in those files looked complicated, but I haven't checked all details. |
I've left the function pausePermsSave() in the controller for now,
Wasn't sure if we should merge with the other pause functions:
pausePermsSave(), pausAttachmentMaintenance(), pauseRepairProcess()
In the end they all do the same.. stopping a process to avoid overload and continue after a short time..
Hopefully this PR doesn't interfere with @Spuds work on the custom profile fields.
http://www.elkarte.net/index.php?topic=291.msg1936#new