Moodle
  1. Moodle
  2. MDL-31147 META: Collect together deprecated code changes for Moodle 2.3
  3. MDL-32942

Deprecated PARAM_ACTION and PARAM_FORMAT should be replaced with PARAM_ALPHANUMEXT

        Details

        • Type: Sub-task Sub-task
        • Status: Closed
        • Priority: Minor Minor
        • Resolution: Fixed
        • Affects Version/s: 2.4
        • Fix Version/s: 2.4
        • Component/s: Libraries
        • Labels:
        • Testing Instructions:
          Hide

          Test 1:
          grep -Ri PARAM_ACTION .
          grep -Ri PARAM_FORMAT .
          and make sure it's not used in code. Only declaration is there in lib/moodlelib.php

          Test 2:
          Review changes and try find any silly mistake.

          Show
          Test 1: grep -Ri PARAM_ACTION . grep -Ri PARAM_FORMAT . and make sure it's not used in code. Only declaration is there in lib/moodlelib.php Test 2: Review changes and try find any silly mistake.
        • Affected Branches:
          MOODLE_24_STABLE
        • Fixed Branches:
          MOODLE_24_STABLE
        • Pull from Repository:
        • Pull Master Branch:
          wip-mdl-32942
        • Pull Master Diff URL:

          Description

          PARAM_ACTION and PARAM_FORMAT were deprecated in 2.0, please replace it with PARAM_ALPHANUMEXT and remove PARAM_ACTION and PARAM_FORMAT declaration.

            Gliffy Diagrams

              Issue Links

              has a non-specific relationship to

              Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-32976 Move deprecated params to deprecatedlib.php and add testcases for them.

              • Minor - Minor loss of function where workaround is possible
              • Open
              has been marked as being related by

              Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-30651 Deprecated PARAM_INTEGER should be replaced with PARAM_INT

              • Minor - Minor loss of function where workaround is possible
              • Closed

              Sub-task - Issues are sometimes broken into multiple sub-tasks. MDL-32940 Deprecated PARAM_NUMBER should be replaced with PARAM_FLOAT

              • Minor - Minor loss of function where workaround is possible
              • Closed

                Activity

                Ascending order - Click to sort in descending order
                Hide
                Permalink
                Rajesh Taneja added a comment -

                Not removed declaration for 3rd party compatibility reasons.

                Show
                Rajesh Taneja added a comment - Not removed declaration for 3rd party compatibility reasons.
                Hide
                Permalink
                Sam Hemelryk added a comment -

                Hi Raj,

                Changes look good thanks, feel free to put this up for integration when ready.

                There are two thoughts I've had while looking at these changes.

                1. We should consider moving the deprecated params to deprecatedlib.php and giving them all an @since version.
                2. We should add a unit test to tests/moodlelib_test.php to test all deprecated PARAM_* are matching to the expected type (just to make sure no-one makes further changes and to be truely aware when removing them for good)

                Cheers
                Sam

                Show
                Sam Hemelryk added a comment - Hi Raj, Changes look good thanks, feel free to put this up for integration when ready. There are two thoughts I've had while looking at these changes. We should consider moving the deprecated params to deprecatedlib.php and giving them all an @since version. We should add a unit test to tests/moodlelib_test.php to test all deprecated PARAM_* are matching to the expected type (just to make sure no-one makes further changes and to be truely aware when removing them for good) Cheers Sam
                Hide
                Permalink
                Rajesh Taneja added a comment -

                Thanks Sam,

                Is it fine to open a new bug to move deprecated params and add test case or shall I do it as part of this bug?

                Show
                Rajesh Taneja added a comment - Thanks Sam, Is it fine to open a new bug to move deprecated params and add test case or shall I do it as part of this bug?
                Hide
                Permalink
                Sam Hemelryk added a comment -

                Probably best to handle it as a separate bug, and I'd both move the params and create test cases at the same time.
                Before you do however check with Petr, just to be sure there is no good reason to keep them in the current location. I think moving them is a good idea that way when people locate them to check how they work they will find them in deprecated lib and be less tempted to use them.

                Show
                Sam Hemelryk added a comment - Probably best to handle it as a separate bug, and I'd both move the params and create test cases at the same time. Before you do however check with Petr, just to be sure there is no good reason to keep them in the current location. I think moving them is a good idea that way when people locate them to check how they work they will find them in deprecated lib and be less tempted to use them.
                Hide
                Permalink
                Rajesh Taneja added a comment -

                Thanks Sam.

                Show
                Rajesh Taneja added a comment - Thanks Sam.
                Hide
                Permalink
                Dan Poltawski added a comment -

                Delaying till the start of the 2.4 dev cycle. We don't want any more big changes for now.

                Show
                Dan Poltawski added a comment - Delaying till the start of the 2.4 dev cycle. We don't want any more big changes for now.
                Hide
                Permalink
                Rajesh Taneja added a comment -

                Branch re-based
                Pushing for integration review.

                Show
                Rajesh Taneja added a comment - Branch re-based Pushing for integration review.
                Hide
                Permalink
                Dan Poltawski added a comment -

                The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week.

                TIA and ciao

                Show
                Dan Poltawski added a comment - The main moodle.git repository has just been updated with latest weekly modifications. You may wish to rebase your PULL branches to simplify history and avoid any possible merge conflicts. This would also make integrator's life easier next week. TIA and ciao
                Hide
                Permalink
                Rajesh Taneja added a comment -

                Branch re-based.

                Show
                Rajesh Taneja added a comment - Branch re-based.
                Hide
                Permalink
                Aparup Banerjee added a comment -

                Hi Raj, i've also noticed PARAM_MULTILANG and PARAM_CLEANFILE marked as deprecated. (seems since 2.0 , 2009-ish) i'm not sure if there are issues for these , but they could all be linked up somewhere.

                Show
                Aparup Banerjee added a comment - Hi Raj, i've also noticed PARAM_MULTILANG and PARAM_CLEANFILE marked as deprecated. (seems since 2.0 , 2009-ish) i'm not sure if there are issues for these , but they could all be linked up somewhere.
                Hide
                Permalink
                Aparup Banerjee added a comment -

                Thanks Raj, thats integrated into master now.

                ps: i can't see that any docs would require updating.

                pps (Tester): a real tester would run through all the changed pages and verify that the params still work fine. That would be a large test though. your choice is : a developer's hat or a tester's hat? :-p

                Show
                Aparup Banerjee added a comment - Thanks Raj, thats integrated into master now. ps: i can't see that any docs would require updating. pps (Tester): a real tester would run through all the changed pages and verify that the params still work fine. That would be a large test though. your choice is : a developer's hat or a tester's hat? :-p
                Hide
                Permalink
                Ankit Agarwal added a comment -

                Developer's hat cause am a developer

                Everything looks good to me.

                Thanks

                Show
                Ankit Agarwal added a comment - Developer's hat cause am a developer Everything looks good to me. Thanks
                Hide
                Permalink
                Aparup Banerjee added a comment -

                yay, it works!

                This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week.

                Thank you all for taking the time to get us here.

                cheers!

                Show
                Aparup Banerjee added a comment - yay, it works! This issue has been put through rigorous processes and finally swam upstream along with some 65 others this week. Thank you all for taking the time to get us here. cheers!

                  People

                  • Votes:
                    0 Vote for this issue
                    Watchers:
                    0 Start watching this issue

                    Dates

                    • Created:
                      Updated:
                      Resolved: