Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Does not delete videos from content folder #4759

Closed
Odia1 opened this issue Dec 15, 2015 · 14 comments
Closed

Does not delete videos from content folder #4759

Odia1 opened this issue Dec 15, 2015 · 14 comments
Assignees
Labels
Milestone

Comments

@Odia1
Copy link

Odia1 commented Dec 15, 2015

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
Copy link
Contributor

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

@MCGallaspy MCGallaspy added the bug label Dec 15, 2015
@Odia1
Copy link
Author

Odia1 commented Dec 15, 2015

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

@dakkad
Copy link

dakkad commented Dec 16, 2015

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

@MCGallaspy
Copy link
Contributor

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
Copy link
Contributor

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
Copy link
Author

Odia1 commented Dec 18, 2015

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 [email protected]
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
#4759 (comment)
.

@MCGallaspy MCGallaspy added this to the 0.16.0 milestone Dec 18, 2015
@MCGallaspy
Copy link
Contributor

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
Copy link

dakkad commented Dec 18, 2015

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

@dakkad
Copy link

dakkad commented Dec 18, 2015

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
Copy link
Contributor

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

Yes.

@rtibbles
Copy link
Member

rtibbles commented Feb 1, 2016

@radinamatic Please verify that this is fixed in 0.16.

@Odia1
Copy link
Author

Odia1 commented Feb 1, 2016

Will do

Darshan's phone
On Feb 1, 2016 3:12 PM, "Richard Tibbles" [email protected] 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
#4759 (comment)
.

@MCGallaspy MCGallaspy self-assigned this Feb 8, 2016
@radinamatic
Copy link
Member

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
Copy link
Member

Confirming this is fixed in 0.16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants