edx-platform icon indicating copy to clipboard operation
edx-platform copied to clipboard

DeprecationWarning: .text & .listdir are deprecated

Open irtazaakram opened this issue 1 year ago • 3 comments

.text() is deprecated use read_text instead.

Reference: https://github.com/jaraco/path/blob/34634dc494d8385c0b40a3bedcce5458983b3a5d/path/init.py#L869

https://github.com/openedx/edx-platform/blob/15e2834825d4354b9e71e761b95fdcdffa1d1015/common/djangoapps/xblock_django/tests/test_commands.py#L67


.listdir is deprecated; use iterdir instead.

Reference: https://github.com/jaraco/path/blob/34634dc494d8385c0b40a3bedcce5458983b3a5d/path/init.py#L557

https://github.com/openedx/edx-platform/blob/15e2834825d4354b9e71e761b95fdcdffa1d1015/common/djangoapps/xblock_django/translation.py#L104

irtazaakram avatar Jun 27 '24 11:06 irtazaakram

Hi @irtazaakram please assign me , want to work on it.

coderight1 avatar Jun 28 '24 01:06 coderight1

@coderight1 Can you please update the status? are you working on it?

farhan avatar Aug 29 '24 10:08 farhan

Hi @farhan, you can work on it.

coderight1 avatar Aug 29 '24 12:08 coderight1

Hello @feanil @irtazaakram is this issue still open?? Can I work on this issue?

And I'm quite new to open source so if I breach any common practices or make any mistakes, please correct me. I'm here to learn and add value to this repo.

Mritunjaii avatar Sep 12 '24 08:09 Mritunjaii

@Mritunjaii Yes its still open, you can work on it. It's very basic so will be a good task for you to start

farhan avatar Sep 12 '24 10:09 farhan

@farhan @irtazaakram I have made the changes,please review them and update me

Mritunjaii avatar Sep 12 '24 12:09 Mritunjaii

@farhan @irtazaakram I have made the changes,please review them and update me

image

Mritunjaii avatar Sep 12 '24 16:09 Mritunjaii

This issue is fixed into this PR: https://github.com/openedx/edx-platform/pull/36699

dwong2708 avatar May 09 '25 22:05 dwong2708