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

LatLong field on Database cannot be propperly searched

    XMLWordPrintable

    Details

    • Testing Instructions:
      Hide
      1. Create a database with a text field and a non required latlong field.
      2. Add one entry with latlong field empty and one filled.
      3. On list view, check "Advanced search" and just "Save settings".
      4. Check the list is of records is not empty.
      Show
      Create a database with a text field and a non required latlong field. Add one entry with latlong field empty and one filled. On list view, check "Advanced search" and just "Save settings". Check the list is of records is not empty.
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE, MOODLE_34_STABLE
    • Fixed Branches:
      MOODLE_32_STABLE, MOODLE_33_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-59296-master
    • Pull Master Diff URL:

      Description

      If one entry leaves a latlong field empty, the advanced search won't work.

      The reason is that the select has an empty option but the value is not empty is: "," So it will always search for that value and it won't find nothing. Otherwise currently this field should be always filled because there's no empty option.

        Attachments

          Activity

            People

            Assignee:
            pferre22 Pau Ferrer
            Reporter:
            pferre22 Pau Ferrer
            Peer reviewer:
            Juan Leyva
            Integrator:
            Eloy Lafuente (stronk7)
            Tester:
            Ankit Agarwal
            Participants:
            Component watchers:
            Adrian Greeve, Jake Dallimore, Mathew May, Mihail Geshoski, Peter Dias, Sujith Haridasan
            Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:
              Fix Release Date:
              10/Jul/17