We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 1764
    • 680 Posts
    Quote from: R.S. at Dec 19, 2005, 07:26 AM

    If I login as a user with access permissions, I’m able to use the manager backend, but on the front-end, no QE shows up!!
    Only when I login as admin, then the QE shows up...

    Hmmm, I can’t seem to reproduce it. Can someone else give this a try and see if they have the same problem?

    Remon, please make sure your user’s role has permissions to Edit a document, Save Documents, and Run module.

    If this is a common problem then it’s definitely something we’ll need to get taken care of before 0.9.1
      • 24253
      • 125 Posts
      Hi,

      "Run module" was set to off, now it works fine!

      Is it a new option? Didn’t notice it before...

      Thanks for the quick reply!

      Remon
        • 1764
        • 680 Posts
        Quote from: R.S. at Dec 19, 2005, 08:15 AM

        "Run module" was set to off, now it works fine!

        Is it a new option? Didn’t notice it before...

        I think it’s been around since Modules have, but I’m not sure. It’s a really common mistake to miss that. Most people don’t make the connection that QuickEdit is actually a module.
          • 13577
          • 302 Posts
          Did something happen to the NewsListing snippet? I created a web user, turned on perms, etc. so I could add a blog entry. But every time I try the "Add blog entry" I get a SQL error:

          SQL: 
          SELECT DISTINCT sc.id,sc.pagetitle,sc.description,sc.longtitle 
          FROM `modx609`.modx_site_content sc 
          LEFT JOIN `modx609`.modx_document_groups dg on dg.document = sc.id 
          WHERE sc.parent = '2' 
            AND sc.published=1 
            AND sc.deleted=0 
            AND hidemenu=0AND AND hidemenu=0 
            AND (sc.privateweb=0 
            OR dg.document_group IN (2)) 
          ORDER BY sc.menuindex ASC,sc.pagetitle ASC


          Note the hidemenu=0AND AND hidemenu=0 part. Is this the bizarre result of something else I might have done or can anyone duplicate this problem?

          Also regarding QE:

          Personally, I think the click+hold button = drag, and release=drop is a more intuitive method than click once to move, click again to drop menu.

          Moreover, how do you get the menu to collapse again if you choose not to actually select a menu item!!
            Standard Disclaimer
            I could be totally wrong.
            • 1764
            • 680 Posts
            Quote from: jaredc at Dec 19, 2005, 08:51 AM

            Personally, I think the click+hold button = drag, and release=drop is a more intuitive method than click once to move, click again to drop menu.

            Hmmm, I never even noticed that. It seems to work both ways for me and I’m using FF 1.5 also. Are you sure it’s not dragging normally for you? The dragging is a script.aculo.us feature and not something I wrote myself, but I might be able to get it to work better if it’s going to be a problem.

            Quote from: jaredc at Dec 19, 2005, 08:51 AM

            Moreover, how do you get the menu to collapse again if you choose not to actually select a menu item!!

            You should be able to click the menu button on the top again to get it to close. I considered auto-closing the menu after clicking something but wasn’t sure it was the best way to handle thinks. At some point I want to get the menu to open persistently between pages.
              • 13577
              • 302 Posts
              The problem is- you reply to fast wink

              By the time I figured out my own idioticy (how ’bout that word), you had replied pointing it out LOL, Not leaving me adquate time to edit my former post before you replied. Ahem... Anyway, here’s my update-

              Throw tomatos this way. rolleyes I see that click and hold does work... doh! And to get rid of the menu- you click the menu title again. While I’m glad it can collapse again, I still wish there were some sort of blur sensitivity so if I clicked outside the menu, it would assume I’m done and collapse. That seems more consistent with GUI convention.
                Standard Disclaimer
                I could be totally wrong.
                • 13577
                • 302 Posts
                OK - I thought the SQL thing was my fault- because it was tripping over a snippet I was updating. But on further inspection, I think I might have found something.

                In document.parser.class.inc.php, line 1218 (getDocumentChildren funciton) it reads:

                if ($where!='') $where .= 'AND '.$where;
                


                I passed along the $where parameter of ’hidemenu=0’. That turns the where clause in the SQL statement to:

                hidemenu=0AND hidemenu=0

                I don’t think that’s the intended result. The problem is that it shouldn’t be concatenated. This works:

                if ($where!='') $where = 'AND '.$where;
                


                Note the "=" rather than the ".=". This would seem to work, even in multiple clause environments like ’hidemenu=0 AND this=that AND (X = Y OR A=B)’ etc.

                Thoughts?
                  Standard Disclaimer
                  I could be totally wrong.
                  • 18397
                  • 3,250 Posts
                  Just for clarification, I believe you are talking about NewsPublisher right?
                    • 13577
                    • 302 Posts
                    Correct. But that’s actually not even relevant at this point. laugh That was just a red herring on my bug fixing quest.
                      Standard Disclaimer
                      I could be totally wrong.
                      • 25663 MODX Staff
                      • 12,272 Posts
                      Just finished helping Mark getting his tweaks to Menu4 tweaked out. I think it’s pretty solid now and does in match the current style a bit. I did have to modify and clean out some stuff in both the XHTML and the CSS (lots of stuff removed there!) to get it to render cross platform (according to Zi). I need to coordinate with him to get all the files committed and udpated for the default and the MODx menus...

                      Speaking of which, is there any reason to have two sets since they seem to be identical for the most part?
                        Ryan Thrash, MODX Co-Founder
                        Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me