Details

      Description

      we had an issue with the matrix datatype in eZ 4.0.1
      if you create a matrix table then the matrix name of the second table is always wrong.
      its written "Col_2" instead of the real name. this is because of:
      kernel/classes/datatypes/ezmatrix/ezmatrix.php (Line 876)

      original(bug):

      if( $column != null && $column['identifier'] != 'col_'. $i+1 )

      our current(fixed):

      if( $column != null && $column['identifier'] != 'col_'. ($i+1) )

      patch file attached.

      Steps to reproduce
      • create new class
      • add matrix datatype to that class as attribute
      • create at least 2 colums in that matrix datatype and give proper names
      • publish that class
      • create a new object with your new class
      • add some rows to your matrix attribute
        => you will see the wrong 2nd column name
      1. ezmatrix.patch
        0.6 kB
        (inactive) Gunnstein Lye

        Activity

        Hide
        (inactive) Gunnstein Lye added a comment -

        The . and + operators have the same precedence, so it is correct to enforce precedence with parentheses here.

        http://php.net/manual/en/language.operators.precedence.php

        Show
        (inactive) Gunnstein Lye added a comment - The . and + operators have the same precedence, so it is correct to enforce precedence with parentheses here. http://php.net/manual/en/language.operators.precedence.php
        Hide
        (inactive) Gunnstein Lye added a comment -

        Fixed in
        trunk (4.2.0Alpha1) rev. 23700
        stable/4.1 (4.1.4) rev. 23701
        stable/4.0 (4.0.7) rev. 23702

        Show
        (inactive) Gunnstein Lye added a comment - Fixed in trunk (4.2.0Alpha1) rev. 23700 stable/4.1 (4.1.4) rev. 23701 stable/4.0 (4.0.7) rev. 23702
        Hide
        ezrobot added a comment -

        This issue has been automatically closed due to the lack of activity over a long period of time. It is very likely that it is obsolete, but if you think it is still valid, do not hesitate to reopen it and mention why.

        Show
        ezrobot added a comment - This issue has been automatically closed due to the lack of activity over a long period of time. It is very likely that it is obsolete, but if you think it is still valid, do not hesitate to reopen it and mention why.

          People

          • Assignee:
            (inactive) Gunnstein Lye
            Reporter:
            (inactive) Gunnstein Lye
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: