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

Database activity description is visible only in List and Search view

    Details

    • Testing Instructions:
      Hide
      1. Create a database activity
      2. Make sure to add something to the description.
      3. Click "Save and display".
      4. Add some fields
      5. Click on the templates tab and then click save template button.
      6. Click each of the following tabs:
        • "View list"
        • "View single"
        • "Search"
        • "Export"
        • "Templates"
        • "Fields"
        • "Presets"
        • Make sure that the description is below the title and above the tabs on each page.
      Show
      Create a database activity Make sure to add something to the description. Click "Save and display". Add some fields Click on the templates tab and then click save template button. Click each of the following tabs: "View list" "View single" "Search" "Export" "Templates" "Fields" "Presets" Make sure that the description is below the title and above the tabs on each page.
    • Affected Branches:
      MOODLE_24_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      wip-MDL-39040-master

      Description

      The description you enter when creating a database activity is visible when you click on the View List and Search tabs, but is not visible when using any other of the tabs. If a teacher enters detailed directions on how or what to add in an entry, the student will not be able to read them when they actually switch to Add Entry. Right now the only work around would be to duplicate the directions onto the Add Entry template.

        Gliffy Diagrams

          Activity

          Hide
          alrachels AL Rachels added a comment -

          It seems that it takes only one line of code to make the description visible when you click on the Add Entry tab. The line of code that needs to be added goes in:

          moodle/mod/data/edit.php

          echo $OUTPUT->box(format_module_intro('data', $data, $cm->id), 'generalbox', 'intro');

          Add the new line of code after line 252 for Moodle 2.2, 2.3, and 2.4. Add it after line 253 for Moodle 2.5. I have tried it in all four of these versions, and they all seem to work fine.

          Show
          alrachels AL Rachels added a comment - It seems that it takes only one line of code to make the description visible when you click on the Add Entry tab. The line of code that needs to be added goes in: moodle/mod/data/edit.php echo $OUTPUT->box(format_module_intro('data', $data, $cm->id), 'generalbox', 'intro'); Add the new line of code after line 252 for Moodle 2.2, 2.3, and 2.4. Add it after line 253 for Moodle 2.5. I have tried it in all four of these versions, and they all seem to work fine.
          Hide
          rwijaya Rossiani Wijaya added a comment -

          Hi Adrian,

          The patch looks great.

          It would be great if you could also added to export, templates, fields and presets pages. That way it will keep the consistency for the pages.

          Thanks for fixing this issue.

          Rosie

          Show
          rwijaya Rossiani Wijaya added a comment - Hi Adrian, The patch looks great. It would be great if you could also added to export, templates, fields and presets pages. That way it will keep the consistency for the pages. Thanks for fixing this issue. Rosie
          Hide
          abgreeve Adrian Greeve added a comment -

          Thanks Rosie,

          I added the description to each page.

          Submitting for integration.

          Show
          abgreeve Adrian Greeve added a comment - Thanks Rosie, I added the description to each page. Submitting for integration.
          Hide
          poltawski Dan Poltawski added a comment -

          Hi Adrian,

          This doesn't seem like a bug to me, unless i'm missing something, this has been like this since the beginning of time. Its also a ui_change, so i'm holding this for release.

          Feel free to campaign (to martin etc) to get this pushed into the release.

          Show
          poltawski Dan Poltawski added a comment - Hi Adrian, This doesn't seem like a bug to me, unless i'm missing something, this has been like this since the beginning of time. Its also a ui_change, so i'm holding this for release. Feel free to campaign (to martin etc) to get this pushed into the release.
          Hide
          abgreeve Adrian Greeve added a comment -

          Hi Dan,

          No rush. I'm happy to wait until until after the release of 2.5.

          Show
          abgreeve Adrian Greeve added a comment - Hi Dan, No rush. I'm happy to wait until until after the release of 2.5.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Adrian this has been integrated now.

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Adrian this has been integrated now.
          Hide
          fred Frédéric Massart added a comment -

          Passing, thanks! (The description also appeared in 'Add entry').

          Show
          fred Frédéric Massart added a comment - Passing, thanks! (The description also appeared in 'Add entry').
          Hide
          marina Marina Glancy added a comment -

          Thanks for your awesome work! This has now become a part of Moodle.

          Closing as fixed!

          Show
          marina Marina Glancy added a comment - Thanks for your awesome work! This has now become a part of Moodle. Closing as fixed!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13