dashCommerce

Did 2.2 used to resize images? 

System.Threading.ThreadAbortException: Thread was being aborted.
   at System.Threading.Thread.AbortInternal()
   at System.Threading.Thread.Abort(Object stateInfo)
   at System.Web.HttpResponse.End()
   at System.Web.HttpResponse.Redirect(String url, Boolean endResponse)
   at System.Web.HttpResponse.Redirect(String url)
   at MettleSystems.dashCommerce.Web.admin.rolelist.btnSave_Click(Object sender, EventArgs e) in C:\Users\entwickler\Desktop\dashCommerce%203.0%20Beta\dashCommerce 3.0 Beta\Web\admin\rolelist.aspx.cs:line 148

2.2 had fixed image width and height in the UI

8. Visiting a product page that does not exist displays an empty product page and logs a "System.DivideByZeroException: Attempted to divide by zero exception" Since the fix for this is allot bigger than my usual ones i've moved it to a seperet message. My fix is to displays the most popular products instead and a friendly message.

 

Step 1: \Web\Localization\labels.resx

Add a new resource called lblNoProduct and value "The product you are visiting does no longer exist, You may be interested in the following items:"

Note you can change the value to whatever message you want to display instead. So in code view it should now look like

<data name="lblNoProduct" xml:space="preserve">
    <value>The product you are visiting does no longer exist, You may be interested in the following items:</value>
</data>

 

Step 2:  \Web\product.aspx

 Under  

<div id="threeColumnMainContent">

Put

  <div id="divProductSummary" class="productSummary" runat="server">

 And near the bottom around line 121 everything after the </table> replace with

</table>

    </div>

      <asp:Panel ID="pnlNoProduct" runat="server" Visible="false">
        <asp:Label ID="lblNoProduct" runat="server" CssClass="label" /><br /><br />
        <hr />
        <dashCommerce:CatalogList ID="catalogList" runat="server" />
      </asp:Panel>
    </div>
</asp:Content>

 

Step 3: \Web\product.aspx.cs Replace the Page_Load method with this one.

    protected void Page_Load (object sender, EventArgs e) {
      try {
        productId = Utility.GetIntParameter("pid");
        _product = new Product(productId);
        if (_product.BaseSku != null)
        {
            SetProductProperties();
            LogProductBrowse();
            LoadProduct();
            LoadPageTitle();
            LoadProductImages();
            LoadDescriptors();
            LoadReviews();
            LoadCrossSells();
            if (User.Identity.IsAuthenticated)
            {
                //only allow them to review if they haven't reviewed before (select even the non-approved reviews)
                if (_product.ReviewRecords().Find(delegate(Review review) { return review.CreatedBy == WebUtility.GetUserName(); }) == null)
                {
                    pnlReview.Visible = true;
                }
            }
        }
        else
        {
            divProductSummary.Visible = false;
            pnlAddToCart.Visible = false;
            pnlNoProduct.Visible = true;
            LocalizationUtility.AddLabelText(lblNoProduct);
            Page.Title = Master.SiteSettings.SiteName;
            ProductCollection productCollection;
            if (this.Master.SiteSettings.CollectBrowsingProduct)
            {
                productCollection = new ProductController().FetchMostPopularProducts();
            }
            else
            {
                productCollection = new ProductController().FetchRandomProducts();
            }
            catalogList.ProductCollection = productCollection;
        }
      }
      catch (Exception ex) {
        Logger.Error(typeof(product).Name + ".Page_Load", ex);
      }
    }

 

STEP 4: \Web\product.aspx.cs in the LoadProduct method around like 263 remove or comment out the line bellow

_product = new Product(productId);

Hye ASI, how do you have fixed the bug of the "Proceed to ckeckOut" with the shipping provider ?

 

thanks!

I actually just made the save to my general settings.  I could come up with a work around but I am sure that will be fixed in the an upcoming release.

I am still getting the circular error on product. Did I get the fix in the wrong place.

protected void btnSave_Click(object sender, EventArgs e) {

try {

Category category;

int parentId = 0;

int categoryId = 0;

int.TryParse(lblCategoryId.Text, out categoryId);

if(categoryId > 0) {category = new Category(categoryId);

}

else {

category = new Category();

}

category.Name = txtName.Text.Trim();

int.TryParse(ddlParentCategory.SelectedValue, out parentId);

 

if (parentId != category.CategoryId)

category.ParentId = parentId;

category.ImageFile = txtImageFile.Text.Trim();

category.Description = txtDescription.Text.Trim();

category.Save(WebUtility.GetUserName());

Reset();

GetCategoryDataSet();

LoadTreeView(ds);

LoadParentCategoryDropDown(ds);

Master.MessageCenter.DisplaySuccessMessage(LocalizationUtility.GetText("lblProductCategorySaved"));

}

catch(Exception ex) {

Logger.Error(typeof(categoryedit).Name + ".btnSave_Click", ex);

Master.MessageCenter.DisplayCriticalMessage(LocalizationUtility.GetCriticalMessageText(ex.Message));

}

}

#endregion

The fix only stopes you from inserting a product with a circular reference you still need to fix the ones you already added from the DB. You can either remove them from the database or run this sql to fix them.

DB Fix: Run this SQL to fix the dodgy rows

UPDATE dbo.dashCommerce_Store_Category
SET ParentId = 0
WHERE CategoryId = ParentId

Sorry, please ignore the following:  It was my error and not a bug!!!

I would like to add a bug:

 Using PayPal Standard as Payments provider returns error 4020 from PayPal when paypal redirects the user to yoursite/paypal/pdthandler.aspx, and as a result, the user is never redirected to /receipt.aspx to show him that his payment and purchase has been successful:

When PayPalService.Synchronize is called, the message back from PayPal is invariable a fail, because the wrong value is passed into the PdtId paramater.

 Instead of passing in the Identity Token, as specified under YourPayPalMerchantAccount >> Profile >> Website Payments Preferences >> Payment Data Transfer, what gets passed instead is the PayPal API Identity

PayPalService.cs is passed the wrong parameter into its constructor for PdtId.  What gets passed instead is the API Signature for your Merchant Account.  The two things are quite distinct.

As there is no way to store the PDT Identity Token using the Control Panel, I have hardcoded this into the constructor of the PayPalServices.cs class.  Therefore, resolution of this issue would, I presume, require:

1.  Adding a node and writing that value into the web.config

2.  Reading the PDT Identity Token into the PayPalPaymentStandardProvider when it is instantiated.

3.  Passing the correct value, ie, the PDT Identity Token, into the constructor of PayPalService.cs so that its PdtId property holds the correct value.

If an alternative process already exists, and I have overlooked it, please document it more clearly.

If someone from the dev team wants to contact me, re reproducing the error, I would be happy to oblige.

 

 

The PDTID is stored in the database.   It's in DashCommerce_Core_ConfigurationData under the name "paymentServiceSettings".  It's all XML in there.   I know you configure that information through the ADMIN interface.   Now it looks like the Paypal Standard code handling the transaction isn't accessing that information at present, so that's what functionality needs to be added.

 

aewrigley:

As there is no way to store the PDT Identity Token using the Control Panel, I have hardcoded this into the constructor of the PayPalServices.cs class.  Therefore, resolution of this issue would, I presume, require:

1.  Adding a node and writing that value into the web.config

2.  Reading the PDT Identity Token into the PayPalPaymentStandardProvider when it is instantiated.

3.  Passing the correct value, ie, the PDT Identity Token, into the constructor of PayPalService.cs so that its PdtId property holds the correct value.

If an alternative process already exists, and I have overlooked it, please document it more clearly.

 

Please ignore:  This is not a bug, by my misunderstanding of PayPal documentation.

 Thanks for that detail.  However, in dashCommerce 3.0 Beta, I see nowhere in the admin control panel to set the PdtId property.  So maybe this needs adding or I am blind.

I will take a look at the table you mention and see what is in there.

 Thanks

aewrigley:

Thanks for that detail.  However, in dashCommerce 3.0 Beta, I see nowhere in the admin control panel to set the PdtId property.  So maybe this needs adding or I am blind.

I will take a look at the table you mention and see what is in there.

 Thanks

 

ADMIN->CONFIGURATION->PAYMENT CONFIGURATION->CONFIGURE PAYMENT PROVIDERS->PAYPALSTANDARDPAYMENTPROVIDER (Click SET PROVIDER)

 

Then Business Email, PDT ID and IS LIVE are configurable below.

 

 

Oh, &$%##!!, what a blithering idiot, I had initially tested using payments pro, then just copied the PayPal API signature over to the payment standard provider, and didn't read the label.

OK, so it DOES do what it says on the tin, and I have been the victim of PayPal's rotten documentation (I might be an ape, but at least I can try to blame someone else).

I think my posts should be deleted, to avoid confusion, or maybe they document a possible ape trap.

1. A user can review a product twice, or any number of times, until they navigate to a different page.

2.  In the control panel, I can't see there is a way to delete reviews, only 'disapprove' them.