Bug 21855

Summary: Remove deprecated delete_unverified_opac_registrations.pl cronjob
Product: Koha Reporter: Katrin Fischer <katrin.fischer>
Component: Command-line UtilitiesAssignee: Olivia <olivialokm>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: fridolin.somers, gmcharlt, lucas, martin.renvoize, nick, robin
Version: MainKeywords: Academy
Hardware: All   
OS: All   
See Also: https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=20856
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=22580
Change sponsored?: --- Patch complexity: Small patch
Documentation contact: Documentation submission:
Text to go in the release notes:
The functionality of delete_unverified_opac_registrations.pl was moved into the cleanup_database.pl cronjob. Please make sure to adjust your conjob configuration accordingly.
Version(s) released in:
Bug Depends on: 13049    
Bug Blocks:    
Attachments: Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"

Description Katrin Fischer 2018-11-17 21:23:29 UTC
System preference now reads:

opac.pref#PatronSelfRegistrationVerifyByEmail# <br><strong>NOTE:</strong> This system preference requires the <code>misc/cronjobs/delete_unverified_opac_registrations.pl</code> cronjob. Ask your system administrator to schedule it.

I think this cronjob has been deprecated in favor of cleanup_database.

We should remove the deprecated Cronjob next release and change the pref description.
Comment 1 Olivia 2019-01-16 04:02:52 UTC
Created attachment 84041 [details] [review]
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"

To Find Fix:

1. Go to Koha Administration
2. Search PatronSelfRegistrationVerifyByEmail
3. Verify that NOTE now says "system preference requires the
cleanup_database ..."
Comment 2 Olivia 2019-01-16 04:08:48 UTC
Created attachment 84044 [details] [review]
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"

To Find Fix:

1. Go to Koha Administration
2. Search PatronSelfRegistrationVerifyByEmail
3. Verify that NOTE now says "system preference requires the
cleanup_database ..."
Comment 3 Katrin Fischer 2019-01-16 21:10:05 UTC
Created attachment 84095 [details] [review]
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"

To Find Fix:

1. Go to Koha Administration
2. Search PatronSelfRegistrationVerifyByEmail
3. Verify that NOTE now says "system preference requires the
cleanup_database ..."

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Comment 4 Katrin Fischer 2019-01-16 21:13:29 UTC
This went a little further in removing the old script as well. I've looked it up - we added the deprecation warning on 2015-07-28. So I think it's about time. 

Olivia, if you'd like to, it would be nice if you could add a note to the "Text to go in the release notes" field describing the change (old script removed, please use... or something like that)
Comment 5 Mikaƫl Olangcay Brisebois 2019-01-18 20:59:39 UTC
Created attachment 84225 [details] [review]
Bug 21855: Changed "deprecated delete_unverified_opac_registrations.pl cronjob" to "cleanup_database"

To Find Fix:

1. Go to Koha Administration
2. Search PatronSelfRegistrationVerifyByEmail
3. Verify that NOTE now says "system preference requires the
cleanup_database ..."

Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Mikael Olangcay Brisebois <mikael.olangcay-brisebois@inlibro.com>


 Current status: Signed Off
Comment 6 Katrin Fischer 2019-01-19 21:09:03 UTC
Moving to passed QA since Mikael has signed off and freed my QA token :)
Comment 7 Nick Clemens 2019-01-25 17:58:25 UTC
Awesome work all!

Pushed to master for 19.05
Comment 8 Martin Renvoize 2019-01-31 16:41:56 UTC
Pushed to 18.11.x for 18.11.03
Comment 9 Lucas Gass 2019-02-01 22:26:14 UTC
pushed to 18.05.x for 18.05.09
Comment 10 Fridolin Somers 2019-02-04 12:46:25 UTC
The cronjob delete_unverified_opac_registrations.pl has been deprecated in favor of cleanup_database by Bug 13049
Comment 11 Fridolin Somers 2019-02-04 12:48:21 UTC
I prefer not to apply to 17.11.x since its oldoldstable