learningequality / ka-lite

KA Lite: lightweight web server for serving core Khan Academy content (videos and exercises) without needing internet connectivity
https://learningequality.org/ka-lite/
Other
458 stars 304 forks source link

Does not delete videos from content folder #4759

Closed Odia1 closed 8 years ago

Odia1 commented 8 years ago

I would like to trim the video content size by removing certain sections like, say, GMAT. The admin interface seems to allow selecting and asking to delete. But when I check the content folder size it does not shrink, although the admin interface said the videos were deleted. Later restart of ka-lite shows all the videos (the original set without any deletes).

DP

MCGallaspy commented 8 years ago

Hi @Odia1, which version of KA Lite are you using, and what is your operating system?

Odia1 commented 8 years ago

I am using it on Windows-10 . ka-lite version is 0.15.1

dakkad commented 8 years ago

@MCGallaspy I am happy to try and see if I can debug this if you can point me in the right direction?

MCGallaspy commented 8 years ago

Hi @dakkad are you able to replicate @Odia1's error? I would start with that, and I would try to identify which lines of code are responsible for deleting videos -- I can help you with the latter task.

MCGallaspy commented 8 years ago

Just a guess, but maybe this LOC fails, raises an exception, and then the exception is ignored. :(

If you're able to replicate the original error, I would start by either setting a breakpoint there or removing the try-except block (so that the exception isn't squelched) and seeing if that LOC does indeed raise an exception.

Odia1 commented 8 years ago

Hi Guys, I have another question/issue: Can I add a second administrator? Is there anyway for a user to self-change password (I know the admin can change it for a non-admin)?

Thanks

On Wed, Dec 16, 2015 at 1:33 PM, Michael Gallaspy notifications@github.com wrote:

Just a guess, but maybe this LOC https://github.com/learningequality/ka-lite/blob/0.15.x/python-packages/fle_utils/videos.py#L64 fails, raises an exception, and then the exception is ignored. :(

If you're able to replicate the original error, I would start by either setting a breakpoint there or removing the try-except block (so that the exception isn't squelched) and seeing if that LOC does indeed raise an exception.

— Reply to this email directly or view it on GitHub https://github.com/learningequality/ka-lite/issues/4759#issuecomment-165218643 .

MCGallaspy commented 8 years ago

I have another question/issue: Can I add a second administrator?

Yes, run the kalite.bat manage createsuperuser command and follow the prompts. You'll find the kalite.bat script in your /Python27/Scripts directory, wherever you installed Python.

Is there anyway for a user to self-change password

No.

dakkad commented 8 years ago

Sorry for the delay, I will see about getting KA-Lite running on a VM over the weekend.

dakkad commented 8 years ago

Hmm. This could be a bit tricky for me as I am a nix bod with not a lot of experience on Windo$e recently. Is it worth me trying to replicate it on nix?

MCGallaspy commented 8 years ago

Is it worth me trying to replicate it on *nix?

Yes.

rtibbles commented 8 years ago

@radinamatic Please verify that this is fixed in 0.16.

Odia1 commented 8 years ago

Will do

Darshan's phone On Feb 1, 2016 3:12 PM, "Richard Tibbles" notifications@github.com wrote:

@radinamatic https://github.com/radinamatic Please verify that this is fixed in 0.16.

— Reply to this email directly or view it on GitHub https://github.com/learningequality/ka-lite/issues/4759#issuecomment-178193464 .

radinamatic commented 8 years ago

Deleting seems to be working fine on 0.16, no JS or user alert errors, deleted videos non present in content folder.

I'd say it could be closed, but if we are aiming for pre-release this week, let's wait so I can test it with Windows installer! @MCGallaspy

rtibbles commented 8 years ago

Confirming this is fixed in 0.16.