We launched new forums in March 2019—join us there. In a hurry for help with your website? Get Help Now!
MODX Community Forums Archive

Support/Comments for Shopkeeper 0.9-release

    • 34162
    • 1 Posts
    Zombie Reply #1, 16 years, 10 months ago
    This is an auto-generated topic for <a href="http://modxcms.com/extras/package/527">Shopkeeper 0.9-release</a> by andchir.

    Brief Description:
    Whats new in Shopkeeper 0.9:
    1. Made extender for Ditto, which allows you to create widgets for additional parameters goods.
    2. Shopping cart works with the disabled in the browser JavaScript (in part).
    3. Now the selected products are stored in the session, but not in a cookie.
    4. Made record count of goods in stock.
    5. Make a pattern (chunks) to the shopping cart. Now almost everything can be changed and customized.
    6. Product price, taking into account the default settings displayed when the page loads.
    7. Made possible to change the price when you change settings without an index plus.
    8. Made possible to integrate with payment systems.
    9. Translated documentation and examples chunk in English (in part).
    10. Fixed known bugs.
      • 26931
      • 2,314 Posts
      anonymized-26931 Reply #2, 16 years, 10 months ago
      wow - thanks! btw. the link still points to "Shopkeeper 0.9 beta6+-beta"...is the new version already in the repository?

      *edit: aaaah - if you click on "Previous Releases" you’ll see version 0.9
        • 8168
        • 1,118 Posts
        dubbs Reply #3, 16 years, 10 months ago
        Is this a full release yet? or is it still in Beta? Am looking for a tool like this for MODx but keen to have a play only when its ready and all bugs are ironed out.
          • 32004
          • 24 Posts
          wavetraxx Reply #4, 16 years, 10 months ago
          Hi and thank you so much for the new version...
          I have some error, after new install from the beta to the 0.9 Release:

          Following error: 
          Warning: require_once(includes/controls/datasetpager.class.php) [function.require-once]: failed to open stream: No such file or directory in /home/httpd/vhosts/stephanstern.ch/httpdocs/manager/processors/execute_module.processor.php(114) : eval()'d code on line 17

Fatal error: require_once() [function.require]: Failed opening required 'includes/controls/datasetpager.class.php' (include_path='/home/httpd/vhosts/stephanstern.ch/httpdocs/manager/includes/') in /home/httpd/vhosts/stephanstern.ch/httpdocs/manager/processors/execute_module.processor.php(114) : eval()'d code on line 17


          Following line change from 
          require_once "includes/controls/datasetpager.class.php";


          to 
          include_once "./includes/controls/datasetpager.class.php";
          , that works.
            • 8522
            • 145 Posts
            eerne Reply #5, 16 years, 10 months ago
            Hi Andchir

            I noticed Shopkeeper in the German modxcms.de forum [1] and like it a lot so far. Sharkbait has made a nice translation for 0.9-beta6, which would be nice if it is included in the main version.

            I was testing a bit and want to ask you if we can include a parameter to exclude the scripts that go into the head. That way I could make a no-javascript version without modifying the original snippet.

            Maybe something like:

            if (!$excludeJavascript) $modx->regClientStartupHTMLBlock($headHtml);


            What do you think? Let me know if I should send my modification.

            Thank you

            eni

            [1]:
            http://www.modxcms.de/forum/comments.php?DiscussionID=3138&page=1#Item_0
              • 8522
              • 145 Posts
              eerne Reply #6, 16 years, 10 months ago
              @ wavetraxx
              I just updated from 0.9-beta6 and didn’t have this error, when exactly do you get it and in which file did you do the change you propose?

              @ Andchir
              I think I found a bug in the basket if Javascript is turned off. the delete button parameter &n should zero based, but it starts with 1. that way if you have more then 1 product it deletes the wrong item.
              If Javascript is turned on this problem doesn’t exist because it iterates by js and does it’s own index starting with 0.

              a simple fix in class.shopkeeper.php would be to change this on line 188
              $num = $i+1;

              to
              $num = $i;

              OR just change line 199 to
              'index' => $i
                • 8522
                • 145 Posts
                eerne Reply #7, 16 years, 10 months ago
                Quote from: Andchir at Nov 20, 2009, 12:23 AM

                You did not invent this bug? Why I do not see it?

                This bug exist only if you have more then one product in the basket AND if javascript is turned off, with version 0.9-beta6 and 0.9. I think &num should start with zero to delete the correct item.

                BUT it to works right on your demo page http://shopkeeper.wdevblog.net.ru/shopkeeper_demo/primer_2.html
                which version do you have there?

                  • 8522
                  • 145 Posts
                  eerne Reply #8, 16 years, 10 months ago
                  Quote from: Andchir at Nov 20, 2009, 02:02 AM

                  0.9 unchanged. You make a screenshot to see a bug? And let the parameters of your server.
                  Thanks for looking into it Andchir! I found something now. The &n counts from 1 only if the parameter &cartRowTpl is set.

                  If $cartRowTpl is set then you use $i for &n (around on line 213)
                  If $cartRowTpl is NOT set then you use $num for &n (around on line 199)

                  The problem is only when Javascript turned off.
                    • 8522
                    • 145 Posts
                    eerne Reply #9, 16 years, 10 months ago
                    An other small bug if Javascript is turned off, is when friendly alias is off. Note it has two question marks in the delete link of the basket.
                    /index.php?id=32?shk_action=del&n=2

                    That makes the delete button go to 404 page. Maybe the solution is to use $modx->makeUrl(21,’’,’&page=2&cat=dog’) ? I have to do some more testing for that one.

                    eni
                      • 8522
                      • 145 Posts
                      eerne Reply #10, 16 years, 10 months ago
                      A little feature request,

                      Can we have [+currency+] placeholder in cartTpl? We have it for cartRowTpl, but not for cartTpl.

                      Did you think about a &noJavascript parameter, I suggested earlier?

                      Can I send you a patch, my modified file or should I put my solution online somewhere, for example on github?

                      thank you.
                      eni

                      EDIT: now I also’d love to have [+id+] in cartRowTpl smiley