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

Database module allows you to start adding new entries even if the maximum has already been reached

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 1.9.9
    • 1.9.9
    • None
    • MOODLE_19_STABLE
    • MOODLE_19_STABLE

    Description

      -> As admin, among the settings of an instance of database module, I set: Maximum entries = 1
      -> As student I go to add a record to the database and all is fine
      -> Now, always as the same student, I try to add one more record even if Maximum entries = 1
      -> First weird event: The "Add entry" TAB is still visible.
      -> I proceed to add one more record, I get the form, I fill it and I go to submit it.
      -> At the submit time I get the following message: "You have entered the maximum number of entries allowed!"

      As far as I can I would prevent this behavior. Students should be stopped before filling the form, not after!!!!
      (Forms here, may be hundred fields long and I am helping students to waste their time!)
      I find this behavior not too correct.
      Do you agree?

      Attachments

        1. edit.patch
          0.5 kB
        2. tabs.patch
          0.8 kB
        3. edit_newidea.patch
          0.8 kB
        4. edit_m20.patch
          0.6 kB
        5. tabs_m20.patch
          0.8 kB
        6. 20100923-MDL24033-m19-edit.patch
          2 kB
        7. 2010092302-MDL24033-m20-edit.patch
          1 kB

        Issue Links

          Activity

            People

              andreabix Andrea Bicciolo
              daniss Daniele Cordella
              Nobody Nobody
              Amaia Anabitarte, Carlos Escobedo, Laurent David, Mikel Martín Corrales, Sabina Abellan, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:
                8/Jun/10