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

Allow Oracle to perform concatenation of large text columns

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Won't Do
    • Icon: Minor Minor
    • None
    • 4.3
    • Database SQL/XMLDB
    • None
    • MOODLE_403_STABLE

      This is a follow up to MDL-78787 where we implemented the same for SQL Server - see points #2 & #3 in this comment in discussion with stronk7. Fixing this would achieve parity with the MySQL, Postgres & SQL Server

      Copying the error here (when/if this is issue is resolved, we should increase the character size used in the test from 1300):

      1) core\dml_test::test_sql_concat
       dml_read_exception: Error reading from database (ORA-06502: PL/SQL: numeric or value error: character string buffer too small
       ORA-06512: at "MOODLE.MOODLELIB", line 86
       SELECT  MOODLELIB.UNDO_MEGA_HACK(MOODLELIB.TRICONCAT(description, 'harcoded', MOODLELIB.TRICONCAT(:o_param1, :o_param2, ' ')))  FROM t_test_table 
       [array (
         'o_param1' => 123.45,
         'o_param2' => 'test',
       )])
        
       /var/www/html/lib/dml/moodle_database.php:494
       /var/www/html/lib/dml/oci_native_moodle_database.php:277
       /var/www/html/lib/dml/oci_native_moodle_database.php:1182
       /var/www/html/lib/dml/moodle_database.php:1774
       /var/www/html/lib/dml/tests/dml_test.php:4414
       /var/www/html/lib/phpunit/classes/database_driver_testcase.php:143
      

      It looks like a previous attempt was made at achieving the same some time ago in MDL-43816, but that never arrived, and was eventually closed as inactive

            Unassigned Unassigned
            pholden Paul Holden
            Votes:
            2 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.