ecwid [ecwid] Terms Of Service

There are already two projects that implements a shopping cart.

@ecwid - the drag and drop to BAG is fantastic. The ajax interface is not just cool but actually very useful. How long it will be so that Drupal users can just login and use this as a regular module. It should not need single sign on but should be just working the regular way a module works.

Another good point - as far as I tested the demo - it is free from this serious bug #569754: Allow more than 65535 products in cart/order by increasing quantity column sizes from smallint

I don't see that #2 is a valid objection. This module doesn't implement a shopping cart, but integrates a third-party cart into Drupal, similar to the way the http://drupal.org/project/magento module integrates the third-party Magento cart into Drupal.

I haven't looked at or tried the code itself, so I won't comment on that and I'll leave the status as "needs review".

Not so sure about the status, guys, please correct me if I'm wrong.

@ecwid,

Had a look at your module and liked it. Not so sure about the speeds though coz my laptop and Internet connection are preeety slow.

The code between the div tags in ecwid_shopping_cart.module::function ecwid_shopping_cart_showstore() didn't display the shopping cart so I replaced it (with some minor edits, some of which might not be entirely necessary) with Ruslan's suggestion here: http://forum.x-cart.com/showpost.php?s=56593052acdeae8632ddf75110863c37&...

Here's my edited code:

<div>
<script type='text/javascript' src='http://app.ecwid.com/script.js?$store_id'></script>
<script type='text/javascript'> xCategories("style="); </script>
</div>
<div>
<script type='text/javascript' src='http://app.ecwid.com/script.js?$store_id'></script>
<!-- remove layout parameter if you want to position minicart yourself -->
<script type='text/javascript'> xMinicart("style=","layout=attachToCategories"); </script>
</div>
<div>
<script type='text/javascript' src='http://app.ecwid.com/script.js?$store_id'></script>
<script type='text/javascript'> xProductBrowser("itemsPerRow=3","itemsPerPage=6","searchResultsItemsPerPage=10","style="); </script>
<noscript>Your browser does not support JavaScript. Please proceed to <a href="https://app.ecwid.com/jsp/$store_id/catalog">HTML version of Demo store</a></noscript>
</div>

That sorted it out. It works fine now. Hope that helps.

Hmmm, very strange. After restoring the ORIGINAL code, it now works fine when NOT logged in, even on Opera and SeaMonkey!

HOWEVER, strange things do happen when in ADMIN mode. The shopping bag disappears altogether in Firefox, Chrome and Avant (the default behaviour for Opera and Safari last time I tested), and is anchored on the lower right edge of the browser window in Opera, SeaMonkey and Safari.

Switching to the edited code gives the consistent look and feel that's expected . Don't know if that's the same experience others have.

Other than that, no problems encountered this time round. Anyone else?

Drupal site hosted on server with the following specs:

1. Apache version: 2.2.11 (Unix)
2. PHP version: 5.2.9
3. MySQL version: 5.0.85-community
4. Architecture: i686
5. Operating system: Linux

Tested on the following browsers running on Win XP SP3:

1. Opera 10.01
2. Avant 11.7 build 43
3. Firefox 3.5.6
4. Safari 4.0.2
5. IE 7.0.5730.13
6. Google Chrome 4.0.249.78
7. SeaMonkey 2.0.2 - took FOREVER to load but loaded nonetheless :)

The module doesn't include the installation file that should be added to remove the Drupal variables used by the module.

1.   $items['admin/settings/ecwid_shopping_cart'] = array(
       'title' => t('Ecwid Shopping Cart module settings'),
       'description' => t('You can setup your store settings here.'),
       'page callback' => 'drupal_get_form',
       'page arguments' => array('ecwid_shopping_cart_admin'),
       'access arguments' => array('access administration pages'),
       'type' => MENU_NORMAL_ITEM,
     );
   

     if ($op == "list") {
       $block = array();
       $block[0]["info"] = t('Minicart');
       $block[1]["info"] = t('Vertical Categories');
       $block[2]["info"] = t('Search Box');
       return $block;
     }
   

   Menu titles, and descriptions should not be passed to t() as that is already done by Drupal core code.

   Strings used in the user interface should be in sentence case.

2. hook_install() should not be used to set Drupal variables used by the module; the second argument passed to variable_get() is the value returned if the variable has not been yet set.

Excerpt from ecwid's Terms Of Service (http://www.ecwid.com/terms-of-service-html, user conduct and submissions, paragraph2):

"You give CDEV a non-exclusive, free, worldwide license for the duration of the applicable author’s rights, to publish your Content. In addition to the right to publish, you also grant the following rights, without limitation: (i) the right to reproduce; (ii) the right to transfer, which includes the distribution via computer and networks; (iii) the right to edit, modify, adapt, arrange, improve, correct, translate, in all or in part; (iv) the right to update/upgrade by adding or removing; and (v) the right to film, perform or post the Content in any media. Except as described in our Privacy Policy, the Company will not be required to treat any Content as confidential."

Any questions? Forget about your intellectual property rights if you use this "shop".