We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
    • 10666
    • 68 Posts
    Well... thank you for the quick answer laugh .
    Seems like you’ve fixed something and introduces some new bugs tongue .
    Let’s take my previous menu for example... it had 3 levels.
    In 0.8 the links that point to folders are converted to: link/ ...BUT only if they are on the first level of the menu (item x ) , and the subitems , even if they are folders are still with the .html suffix.
    Also , I noticed a regression: now, the URLs of the loaded page(the one in the adress bar) is of page.html form , instead of page/ if they are folders.
    Good luck smiley!
      • 1932
      • 137 Posts
      Oops, try changing the following line:

      $alias = array_search($documentIdentifier, $modx->documentListing);

      to

      $alias = basename(array_search($documentIdentifier, $modx->documentListing));

      I’ve updated the above plugin code and it shooouuld work for you.

      Note: In order to avoid extra DB calls, empty containers will not display as folders. They can, of course, be overridden using the TV option.
        • 23491 ☆ A M B ☆
        • 1,056 Posts
        Sure, point out all my flaws! LoL J/k

        Apoxx,

        In testing v 0.8 Plugin, I am finding that with seoOverride set to "folder" on my Empty container, it still is not forcing "alias/" ... However, the generated links vs. resolved URL are now similar, which is nice!

        NOTE: On www.pixelchutes.com Portfolio IS an empty container, while Contact is normal document.
          Mike Reid - www.pixelchutes.com
          MODx Ambassador / Contributor
          [Module] MultiMedia Manager / [Module] SiteSearch / [Snippet] DocPassword / [Plugin] EditArea / We support FoxyCart
          ________________________________
          Where every pixel matters.
          • 1932
          • 137 Posts
          @Pixelchutes: Interesting, I did some testing and I’m not experiencing this problem. Can you list your plugin’s configuration settings?
            • 10666
            • 68 Posts
            Note: In order to avoid extra DB calls, empty containers will not display as folders. They can, of course, be overridden using the TV option.
            I assume that by empty containers , you mean: containers that have no children.

            That being said , even with the corrected 0.8 version I have some inconsistency issues.I’ve described my problem bellow:
            Item1 (empty container) [ link:    Item1/ ] , [ resolved URL: Item1.html ]
            
            Item2 (container with children) [ link:    Item2/ ] , [ resolved URL: Item2/ ]
            |  subItem1 (container with children) [ link:    subItem2.html ] , [ resolved URL: subItem2/ ]
            |  |  subSubItemX ...
            |
            |  subItem2 (empty container) [ link:    subItem4.html ] , [ resolved URL: subItem4.html ]
            
              • 23491 ☆ A M B ☆
              • 1,056 Posts
              Update RE: v0.8:

              When I have FURLS on, but NO ALIAS defined, usually it would output 2.html, 3.html, etc....

              However, my rendered HTML gets a little "messed up"

              <a /" title="Test Page A" ...
              


              should be something like:

              <a href="2.html" title="Test Page A" ...
              


              ALSO:

              With version 0.7.3 AND 0.8:

              I notice eForm is NOT working ONLY when on the homepage... (Form submitting to itself with ACTION=’[~[*id*]~]’ )

              Because [~[*id*]~] is resolving to index.html, NOT /, and since index.html is 301’d to "/", the form doesn’t submit properly ?? sad

              As it may be related, I should point out that [~[*id*]~] is called from within a Chunk, called by eForm...

              Well, I guess I forgot that it only rewrites HREF=" ... ", but I definitely think ACTION=" ... " should also be included...

                Mike Reid - www.pixelchutes.com
                MODx Ambassador / Contributor
                [Module] MultiMedia Manager / [Module] SiteSearch / [Snippet] DocPassword / [Plugin] EditArea / We support FoxyCart
                ________________________________
                Where every pixel matters.
                • 23491 ☆ A M B ☆
                • 1,056 Posts
                Quote from: ApoXX at Feb 09, 2007, 01:26 AM

                @Pixelchutes: Interesting, I did some testing and I’m not experiencing this problem. Can you list your plugin’s configuration settings?

                Even in v0.8.1, When I have a container (be it empty (no children) or not) with seoOverride, it’s not showing up as "alias/", rather "alias"

                I just copy and paste the plugin from the form, no modification...

                I noticed that $override is set to "0" by default, when I change this to 1, the URL resolves to /alias/, but all links remain "/alias" ....
                  Mike Reid - www.pixelchutes.com
                  MODx Ambassador / Contributor
                  [Module] MultiMedia Manager / [Module] SiteSearch / [Snippet] DocPassword / [Plugin] EditArea / We support FoxyCart
                  ________________________________
                  Where every pixel matters.
                  • 1932
                  • 137 Posts
                  Below is the latest release, which should address most reported issues.

                  Latest release posted further in the thread.
                    • 25663 MODX Staff
                    • 12,272 Posts
                    For the .htaccess changes, what if we want to force all queries to the http://mysite.com/ version vs. http://www.mysite.com/ version?
                      Ryan Thrash, MODX Co-Founder
                      Follow me on Twitter at @rthrash or catch my occasional unofficial thoughts at thrash.me
                      • 10666
                      • 68 Posts
                      Works great Apoxx !
                      I haven’t spotted any bugs ,it looks very good so far .

                      Question: If I still want to have empty containers show as folders,without using TV overrides , would it suffice to supress the
                      count($modx->getChildIds($documentIdentifier, 1)) > 0
                      checks ? Haven’t done that yet,will test it tomorrow ( though having it as a config option would be even better ).