We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 10487 MODX Staff
    • 1,535 Posts
    Jared, I’ve got a fix to commit to sort out that behaviour ... it’s because all the default values for user attributes are being saved when a manager user is saved. It’s best to remove the defaults so only values which have been physically changed by the user are saved.
      Garry Nutting
      Senior Developer
      MODX, LLC

      Email: [email protected]
      Twitter: @garryn
      Web: modx.com
      • 10487 MODX Staff
      • 1,535 Posts
      Quote from: kylej at Oct 05, 2006, 09:40 AM

      Here is one bug I would personally like to see fixed:

      http://modxcms.com/bugs/task/512
      Kyle, I have commited the GROUP BY fix to my SVN branch, it works fine for the sorting problems and nobody who has implemented it has reported any adverse effects so I’m happy it does the job smiley
        Garry Nutting
        Senior Developer
        MODX, LLC

        Email: [email protected]
        Twitter: @garryn
        Web: modx.com
        • 32963
        • 1,732 Posts
        Many thanks Zi,

        I’ve only worked on the MODxLight theme so I have not tested the others as yet. Once the Ligth is working fine with the new code then it should be very easy to fix the others.


        Quote from: zi at Oct 06, 2006, 02:55 AM

        the new MODx Light theme works PERFECT for me smiley ... its faster, the working progress indicators and a tree hide icons are placed right IMO.

        MODx Green seems to have a problem, I m on FF 2.0 RC1 and IE 6.0, Tree Icons are messed up, context menu is not working in both browsers.

        Same for MODx theme.

        I am attaching the screenshots.
          xWisdom
          www.xwisdomhtml.com
          The fear of the Lord is the beginning of wisdom:
          MODx Co-Founder - Create and do more with less.
          • 32963
          • 1,732 Posts
          Quote from: Dimmy at Oct 06, 2006, 03:01 AM

          I was just making screenshots to show the same thinks I noticed that some break character or mabe something else was present in the top left corner I hilighted the space to show also the icons are packt togetter with no H-spacing at all.

          I like this theme. Is this the theme called "MODx" ?
            xWisdom
            www.xwisdomhtml.com
            The fear of the Lord is the beginning of wisdom:
            MODx Co-Founder - Create and do more with less.
            • 32963
            • 1,732 Posts
            Quote from: PaulGregory at Oct 06, 2006, 07:06 AM

            I never switch Captcha on for the Manager. I have no idea how many users out there do use this feature. I’m not mad happy on the default "3 false logins blocks the user for an hour" thing, it strikes me that people could go to /manager on lots of sites and deliberately screw up the predictable "admin" account at crucial moments.

            However, I do like that admins have the choice. I’d rather that they could switch off the blocking, but admins can always change the logins to 60 and the time to 10 minutes or so if they want to rely on Captcha instead.

            So: don’t remove it, but yes it needs to be in a better place than in that picture.

            Hi Paul,

            I don’t think we should allow users to disable login timouts but I think we can add the other options the configuration screen.

            Personally I’ll be add these options to the system configuration and will also add delays to login failure.
              xWisdom
              www.xwisdomhtml.com
              The fear of the Lord is the beginning of wisdom:
              MODx Co-Founder - Create and do more with less.
              • 15987
              • 786 Posts
              Sounds good garry, I wonder if this will error on other versions of mysql though.

              I was under the assumption that if you grouped by a field all other fields had to be calculated fields. Maybe this restriction doesn’t exist in mysql though.

              If it works under all versionas that great though smiley
                • 32963
                • 1,732 Posts
                Quote from: kylej at Oct 06, 2006, 03:26 PM

                Sounds good garry, I wonder if this will error on other versions of mysql though.

                I was under the assumption that if you grouped by a field all other fields had to be calculated fields. Maybe this restriction doesn’t exist in mysql though.

                If it works under all versionas that great though smiley

                In SQL (AFAIK) only non-aggregated columns are listed inside the Group By clause. So if the column is not using one of those aggregated functions then it should be present inside the Group By. This again might have some performance impact on large number of records but I could be wrong here. Not 100% on how MySQL operates in regards to Group By
                  xWisdom
                  www.xwisdomhtml.com
                  The fear of the Lord is the beginning of wisdom:
                  MODx Co-Founder - Create and do more with less.
                  • 7455
                  • 2,204 Posts
                  Quote from: xwisdom at Oct 06, 2006, 03:21 PM

                  Quote from: Dimmy at Oct 06, 2006, 03:01 AM

                  I was just making screenshots to show the same thinks I noticed that some break character or mabe something else was present in the top left corner I hilighted the space to show also the icons are packt togetter with no H-spacing at all.

                  I like this theme. Is this the theme called "MODx" ?

                  Yes it is
                    follow me on twitter: @dimmy01
                    • 10487 MODX Staff
                    • 1,535 Posts
                    Okay, did some research ...


                    In SQL (AFAIK) only non-aggregated columns are listed inside the Group By clause. So if the column is not using one of those aggregated functions then it should be present inside the Group By. This again might have some performance impact on large number of records but I could be wrong here. Not 100% on how MySQL operates in regards to Group By
                    MySQL uses something called ’GROUP BY with hidden fields’ as long as mySQL is not running in ONLY_FULL_GROUP_BY mode which I don’t believe is a common mySQL mode.

                    More info on GROUP BY with hidden fields: http://mysql.speedbone.de/doc/refman/4.1/en/group-by-hidden-fields.html (because we’d be grouping by sc.id, then we will be getting unique fields returned)

                    I suppose, the question is whether we want to include this as a temporary fix in 0.9.5 and investigate better ways to do this (which we’ll have to do at some point as we move down the road of multiple RDBMS support in later versions of MODx), or we leave it out and advise 4.1.21 users to manually patch their installs.

                    Any thoughts?
                      Garry Nutting
                      Senior Developer
                      MODX, LLC

                      Email: [email protected]
                      Twitter: @garryn
                      Web: modx.com
                      • 7455
                      • 2,204 Posts
                      When using the docmanager i wanted to sort the doc tree but when i click on a folder or site root the docmanager disapears and i see the normal preview screen of that folder or the startpage when i click on the siteroot
                        follow me on twitter: @dimmy01

                      This discussion is closed to further replies. Keep calm and carry on.