Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-18694

Problem with borders in tables in ezoe in 2011.6

    Details

    • Type: Bug Bug
    • Status: Open
    • Priority: Medium Medium
    • Resolution: Unresolved
    • Affects Version/s: 2011.6
    • Fix Version/s: None
    • Component/s: Legacy > Extensions
    • Labels:
      None

      Description

      I have found a problem in 2011.6, ezoe 5.40-dev, reediting tables.

      If a table is saved with no border, still it gets a border 1 when loaded into ezoe for reediting.

      Have a look in ezoe/ezxmltext/handlers/input/ezoexmlinput.php around line 1530:

       
      if ( is_string( $border ) ) 
      
      

      I believe this check is not enough, as the $border variable still is a string when empty.

      I changed to the line below, and now it seems to work:

       
       if ( is_string( $border ) && !empty($border) ) 
      
      

      Steps to reproduce

      In 2011.6, create a table with ezoe in any content object.
      Set the border to 0 for the table.
      Now publish the object.
      Then edit the object again.
      Now, notice that in edit mode the border property is now set to 1.

        Activity

        Hide
        Anja Lundin added a comment -

        This bug is not present in version 2012.6. The solution above is implemented as far as I can see.

        Show
        Anja Lundin added a comment - This bug is not present in version 2012.6. The solution above is implemented as far as I can see.
        Hide
        scrieler _ added a comment -

        I can confirm this problem, thanks for the solution!

        Show
        scrieler _ added a comment - I can confirm this problem, thanks for the solution!

          People

          • Assignee:
            unknown
            Reporter:
            Nicklas Lundgren
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: