Delete media not working at this time. Not sure if it ever was.

Joomla-Bible-Study / Proclaim

Joomla Proclaim Component - Christian Web Ministries

https://www.christianwebministries.org/

GNU General Public License v3.0

10 stars 9 forks source link

Delete media not working at this time. Not sure if it ever was. #855

Closed bcordis closed 2 years ago

bcordis commented 3 years ago

Would need to look into

st-anton commented 3 years ago

Hi Brent, I was absent for a long time... I debugged is and found that the admin model/mediafile.php line 741 is called, where state is checked for -2 (line 745). But state is not part of the table variable. It is simply not there. So the call fails. Removing the lines if ($record->state != -2) { return false; } would work. Deleting a study, there is no function canDelete in the studies model. So studies can be deleted.

bcordis commented 3 years ago

@st-anton so glad you have been able to take time to look at the system. I'll try and make a pull request with this change and see if you can look at it also.