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

Paypal Cost Validation

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 2.4.1
    • Fix Version/s: 2.3.8, 2.4.5, 2.5.1
    • Component/s: Enrolments
    • Labels:
    • Testing Instructions:
      Hide

      1/ in 23 and 24 test cost validation when editing PayPal enrol method, only "." is allowed there
      2/ repeat 1/ in 25, "." is allowed together with localised dec separator "," in CZ,DE,FR,SP...
      3/ test cost display when enrolling in 25
      4/ test actual paypal enrolment in 25

      Note: there is no need to test 26

      Show
      1/ in 23 and 24 test cost validation when editing PayPal enrol method, only "." is allowed there 2/ repeat 1/ in 25, "." is allowed together with localised dec separator "," in CZ,DE,FR,SP... 3/ test cost display when enrolling in 25 4/ test actual paypal enrolment in 25 Note: there is no need to test 26
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_23_STABLE, MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      w22_MDL-38408_m26_paypalcost

      Description

      I have been looking at the Paypal cost validation cost. It should not take non-numeric values but ti does on my installation.
      The function to check if a value is numeric is there (edit_form.php) but i do not see where it is being called.

      Is it possible to get advise on how to fix this, please.

        Attachments

          Activity

            People

            Assignee:
            skodak Petr Skoda
            Reporter:
            fikira Lilian
            Integrator:
            Dan Poltawski
            Tester:
            Ankit Agarwal
            Participants:
            Component watchers:
            Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Sara Arjona (@sarjona)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              8/Jul/13