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

Coding error by using Download all file submissions #78

Closed
Clarence71 opened this issue Jan 11, 2024 · 10 comments
Closed

Coding error by using Download all file submissions #78

Clarence71 opened this issue Jan 11, 2024 · 10 comments

Comments

@Clarence71
Copy link

Hi,
we upgraded Moodle to 4.3.2 (from 3.11 over 4.0 ...)
installed Student Folder Plugin 4.2.2 and all works fine except the all file button.
error message: get_all_user_name_fields() is deprecated. Please use the \core_user\fields API instead
any known issues or solutions?
I am writing from Germany
Greetings Lars

@goggo24
Copy link
Member

goggo24 commented Jan 11, 2024

Thanks for reporting. We found the bug too ;) We are currently working on v.4.3.0 and we will publish this version in the next 2 weeks. Is the fix urgent? If yes, we can make our current development branch available here via github, but without garanty. Regards, Eva

@Clarence71
Copy link
Author

Hi Eva,
do you have any (good) news for me?
we would also try developer inputs, if you have some.
Lars

@goggo24
Copy link
Member

goggo24 commented Jan 19, 2024

We fixed the bug already, it will be included in v.4.3.0.

@goggo24
Copy link
Member

goggo24 commented Feb 12, 2024

Since last week v.4.3.0 from publication is available. Have fun!

@goggo24 goggo24 closed this as completed Feb 12, 2024
@wiebkemueller-hsh
Copy link

Dear @goggo24 Eva, will this be backported to 4.1.9? We're still using this LTS Version of Moodle. Thank you for considering! Wiebke

@goggo24
Copy link
Member

goggo24 commented Feb 13, 2024

@wiebkemueller-hsh this error occured as far as we know only under Moodle 4.3.x. I've just tested it on our test environment under 4.1.6+ and 4.2.3 - there it works fine. Can you give me some additional information - which version of publication do you use? If you press the link "Download all files as ZIP" the error occurs? Which mode do you use (upload or assignment)?

@goggo24 goggo24 reopened this Feb 13, 2024
@wiebkemueller-hsh
Copy link

wiebkemueller-hsh commented Feb 13, 2024

Hi @goggo24 thanks for the super quick reply.

We are testing Moodle [4.1.9 (Build: 20240212] today and have installed mod_publication
v4.1.1
2022121201

It happens in both modes, "upload mode" or "assigment mode", when I click on "Download all files as ZIP" I get
get_all_user_name_fields() is deprecated. Please use the \core_user\fields API instead
line 3280 of /lib/deprecatedlib.php: call to debugging()
line 983 of /mod/publication/locallib.php: call to get_all_user_name_fields()
line 129 of /mod/publication/view.php: call to publication->download_zip()

Warning: Cannot modify header information - headers already sent by (output started at /var/www/html/lib/weblib.php:3389) in /var/www/html/lib/filelib.php on line 2399

see also screenshot attached.
1_mod-pub

Thank you
Wiebke

@goggo24
Copy link
Member

goggo24 commented Feb 13, 2024

Okay, thanks for the fast feedback - we will have a look.

@wiebkemueller-hsh
Copy link

Hi Eva,

@goggo24

Just to let you know: For whatever reason the problem didn't occur anymore in our live system that is configured with the same parameters as the test system mentioned above. (?)

Regards!
Wiebke

@goggo24
Copy link
Member

goggo24 commented Feb 21, 2024

@wiebkemueller-hsh good to know ;) As we already mentioned, we couldn't reproduce the bug and don't get the deprecated message nor the warning. We also tried it on another system, but the same behaviour . We are currently checking the code directly. If we find something, we'll let you know, but as it works on your production system, we will not give it a high priority.

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

No branches or pull requests

4 participants