Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-71099

Move user_fields from core to core_user

    XMLWordPrintable

    Details

      Description

      There is a new "utility class" in code (MDL-45242), \core\user_fields for 311 and up.

      After some discussion @ developer chat it was commented that, maybe, it had more sense to make that class part of the user subsystem.

      That will imply moving it to be \core_user\user_fields (or maybe \core_user\fields ??) and, also, switch all the current uses in core to the new class.

      No need for class aliasing should be applied as far as the affected versions are, all them, dev versions.

      So this issue is about:

      1) Decide if we perform the move.
      2) Apply for it if agreed to.

      Ciao

      PS: Also, it was agreed that, in any case, the new class is not an API per se (it's not going to be implemented or extended over other components), so just a nice util class (like core_text and others...).

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              quen Sam Marshall
              Reporter:
              stronk7 Eloy Lafuente (stronk7)
              Peer reviewer:
              David Mudrák (@mudrd8mz) David Mudrák (@mudrd8mz)
              Integrator:
              Eloy Lafuente (stronk7) Eloy Lafuente (stronk7)
              Tester:
              CiBoT CiBoT
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona), Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan, Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                17/May/21

                  Time Tracking

                  Estimated:
                  Original Estimate - 0 minutes
                  0m
                  Remaining:
                  Remaining Estimate - 0 minutes
                  0m
                  Logged:
                  Time Spent - 1 hour
                  1h