-
Notifications
You must be signed in to change notification settings - Fork 468
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
TrackEDownload entity has no $sessionId property #5412
Comments
Internal: Adjust down migration for track_e_downloads schema changes - refs #5412
So for the sake of documentation, the track_e_downloads table has been modified to replace However, I don't see where we are migrating the data in the current migration. |
As @christianbeeznest noted, I might have previously said that this data was not very important, but dropping it is practically removing forcibly information from a database, so I've asked him to add the necessary migration for that data to end up in resource_link. This might add considerable weight to 1.* -> 2.0 migrations, so a likely point of optimization would be to delete the data from |
Ping @christianbeeznest |
Migration: Add TrackEDownloads entries with null resourceLink - refs #5412
In migration Version20220614165020, the down_session_id column is renamed to session_id
chamilo-lms/src/CoreBundle/Migrations/Schema/V200/Version20220614165020.php
Lines 36 to 37 in 75b3424
However, the TrackEDownloads entity has no $sessionId property, down_session_id column is deleted (but it doesn't exists) in the PR #5086 https://github.com/chamilo/chamilo-lms/pull/5086/files#diff-f5716d8b38a2eb470cbff6d387a561bb9122a26f75094dd3c369778148fa2734R21
With a Chamilo migrated, when viewing or downloading a resource file you get the
An exception occurred while executing a query: SQLSTATE[HY000]: General error: 1364 Field 'session_id' doesn't have a default value
messageThe text was updated successfully, but these errors were encountered: