Details

    • Branch Version/s:
      6.1.x
    • Backported to Branch:
      Committed
    • Fix Priority:
      3
    • Similar Issues:
      Show 5 results 

      Description

      When a user creates a theme with configurable settings the settings won't be saved for a Layout with that theme on some circumstances.

      Let's see this theme descriptor file:

      <?xml version="1.0"?>
      <!DOCTYPE look-and-feel PUBLIC "-//Liferay//DTD Look and Feel 6.1.0//EN" "http://www.liferay.com/dtd/liferay-look-and-feel_6_1_0.dtd">
      
      <look-and-feel>
      <compatibility>
      <version>6.1.0+</version>
      </compatibility>
      <theme id="test-theme" name="Test theme">
      <settings>
      <setting configurable="true" type="checkbox" key="test-setting" value="false" />
      </settings> 
      </theme>
      </look-and-feel>
      

      If the user applies this theme on Guest site's public pages and check test-setting checkbox, after saving it we will experience some strange behavior.

      When the user select a child page on Manage Page dialog and select the Look and Feel section he can change the them to be applied this page directly. Note, that the test-setting checkbox will be checked. If the user unchecks it then saves the settings, after the reload of the dialog the checkbox will be checked again.

      The cause of this situation is that the typeSettings propagation to the database is depends on the default value of the theme and selected value. If these values are different, then it will be saved for the page, otherwise it won't be saved.

        Issue Links

          Activity

          Hide
          Michael Saechang added a comment -

          Committed on:
          Portal 6.2.x GIT ID: 14c01a7ad67893d5b3c4c32af529ce4798c8040e.

          Show
          Michael Saechang added a comment - Committed on: Portal 6.2.x GIT ID: 14c01a7ad67893d5b3c4c32af529ce4798c8040e.
          Hide
          Edward Gonzales added a comment -

          Hello everyone! We are in the process of moving component "Theme" to "Theme > Dev" in LPS. Please make the necessary adjustments to your filters. Thanks!

          Show
          Edward Gonzales added a comment - Hello everyone! We are in the process of moving component "Theme" to "Theme > Dev" in LPS. Please make the necessary adjustments to your filters. Thanks!

            People

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

              Dates

              • Created:
                Updated:
                Resolved:
                Days since last comment:
                1 year, 49 weeks, 3 days ago

                Development

                  Structure Helper Panel