OpenCart v1.4.0

MWYS wrote:Just checked it out on my development Pc, looks brilliant.

Love the new front end and especially love the Magento'style backend, its a massive improvement from the last one!

i2Paq wrote:First:
Front and Admin looking great!
The admin is really fast as well.

The Dashboard in the admin is well designed and great to have.

Second:
As I was thinking that we would have Order Edit, Manual Order Entry and a fix for the Invoice-numbers, so I'm a little disapointed.
Adding a new customer still has no options to enter any address etc.

Daniel wrote: This is a quick feature to add so it will be in the next few releases. remember normally I release every 2 weeks. Its only because I spent so much time with the last version that the last release took so long.

Daniel wrote:

i2Paq wrote:First:
Front and Admin looking great!
The admin is really fast as well.

The Dashboard in the admin is well designed and great to have.

Second:
As I was thinking that we would have Order Edit, Manual Order Entry and a fix for the Invoice-numbers, so I'm a little disapointed.
Adding a new customer still has no options to enter any address etc.

This is a quick feature to add so it will be in the next few releases. remember normally I release every 2 weeks. Its only because I spent so much time with the last version that the last release took so long.

Jacob_chan wrote:who can tell me how to install the website? Please help me,please!

Yakiv wrote:Is the cart supposed to update with a new title when the coupon is applied? I see the system message that the coupon was applied, but I don't see the price change. I realize this is probably taken care of in the checkout stage, but it's always good if the coupon starts to do it's job as you are still shopping. Is there any way to get it going, with the same ajax effect as well?

...If you're stuck at work, why don't you honor your employer and do some work! That's what they are paying you for!

Blogexecute wrote:

Daniel wrote:

Blogexecute wrote:Daniel, remove html code in controller files!
remove inline styles!
(will be easier to create new themes)

which inline styles and what html code int he controller?

The first example ? cart.php

Code: Select all

		$output = '<table cellpadding="2" cellspacing="0" style="width: 100%;">';
		
    	foreach ($this->cart->getProducts() as $product) {
      		$output .= '<tr>';
        	$output .= '<td width="1" valign="top" align="right">' . $product['quantity'] . '&nbsp;x&nbsp;</td>';
        	$output .= '<td align="left" valign="top"><a href="' . $this->model_tool_seo_url->rewrite($this->url->http('product/product&product_id=' . $product['product_id'])) . '">' . $product['name'] . '</a>';
          	$output .= '<div>';
            
			foreach ($product['option'] as $option) {
            	$output .= ' - <small style="color: #999;">' . $option['name'] . ' ' . $option['value'] . '</small><br />';
            }
			
			$output .= '</div></td>';
			$output .= '</tr>';
      	}
		
		$output .= '</table>';
    	$output .= '<br />';
    	$output .= '<div style="text-align: right;">' . $this->language->get('text_subtotal') . '&nbsp;' .  $this->currency->format($this->cart->getTotal()) . '</div>';

Others?

Code: Select all

		foreach ($results as $result) {	
			if (!$current_path) {
				$new_path = $result['category_id'];
			} else {
				$new_path = $current_path . '_' . $result['category_id'];
			}
			
			$output .= '<li>';
			
			$children = '';
			
			if ($category_id == $result['category_id']) {
				$children = $this->getCategories($result['category_id'], $new_path);
			}
			
			if ($this->category_id == $result['category_id']) {
				$output .= '<a href="' . $this->model_tool_seo_url->rewrite($this->url->http('product/category&path=' . $new_path))  . '"><b>' . $result['name'] . '</b></a>';
			} else {
				$output .= '<a href="' . $this->model_tool_seo_url->rewrite($this->url->http('product/category&path=' . $new_path))  . '">' . $result['name'] . '</a>';
			}
			
        	$output .= $children;
        
        	$output .= '</li>'; 
		}
 
		if ($results) {
			$output .= '</ul>';
		}

The style inline? Most pages
Example?

Code: Select all

         <td style="text-align: center; width: 250px; vertical-align: top;"><a href="<?php echo $popup; ?>" title="<?php echo $heading_title; ?>" class="thickbox"><img src="<?php echo $thumb; ?>" title="<?php echo $heading_title; ?>" alt="<?php echo $heading_title; ?>" id="image" style="margin-bottom: 3px;" /></a><br />
            <span style="font-size: 11px;"><?php echo $text_enlarge; ?></span></td>
          <td style="padding-left: 15px; width: 296px; vertical-align: top;"><table width="100%">

Greatings

Blogexecute wrote:

Daniel wrote:

Blogexecute wrote:Daniel, remove html code in controller files!
remove inline styles!
(will be easier to create new themes)

which inline styles and what html code int he controller?

The first example ? cart.php

Code: Select all

		$output = '<table cellpadding="2" cellspacing="0" style="width: 100%;">';
		
    	foreach ($this->cart->getProducts() as $product) {
      		$output .= '<tr>';
        	$output .= '<td width="1" valign="top" align="right">' . $product['quantity'] . '&nbsp;x&nbsp;</td>';
        	$output .= '<td align="left" valign="top"><a href="' . $this->model_tool_seo_url->rewrite($this->url->http('product/product&product_id=' . $product['product_id'])) . '">' . $product['name'] . '</a>';
          	$output .= '<div>';
            
			foreach ($product['option'] as $option) {
            	$output .= ' - <small style="color: #999;">' . $option['name'] . ' ' . $option['value'] . '</small><br />';
            }
			
			$output .= '</div></td>';
			$output .= '</tr>';
      	}
		
		$output .= '</table>';
    	$output .= '<br />';
    	$output .= '<div style="text-align: right;">' . $this->language->get('text_subtotal') . '&nbsp;' .  $this->currency->format($this->cart->getTotal()) . '</div>';

Others?

Code: Select all

		foreach ($results as $result) {	
			if (!$current_path) {
				$new_path = $result['category_id'];
			} else {
				$new_path = $current_path . '_' . $result['category_id'];
			}
			
			$output .= '<li>';
			
			$children = '';
			
			if ($category_id == $result['category_id']) {
				$children = $this->getCategories($result['category_id'], $new_path);
			}
			
			if ($this->category_id == $result['category_id']) {
				$output .= '<a href="' . $this->model_tool_seo_url->rewrite($this->url->http('product/category&path=' . $new_path))  . '"><b>' . $result['name'] . '</b></a>';
			} else {
				$output .= '<a href="' . $this->model_tool_seo_url->rewrite($this->url->http('product/category&path=' . $new_path))  . '">' . $result['name'] . '</a>';
			}
			
        	$output .= $children;
        
        	$output .= '</li>'; 
		}
 
		if ($results) {
			$output .= '</ul>';
		}

The style inline? Most pages
Example?

Code: Select all

         <td style="text-align: center; width: 250px; vertical-align: top;"><a href="<?php echo $popup; ?>" title="<?php echo $heading_title; ?>" class="thickbox"><img src="<?php echo $thumb; ?>" title="<?php echo $heading_title; ?>" alt="<?php echo $heading_title; ?>" id="image" style="margin-bottom: 3px;" /></a><br />
            <span style="font-size: 11px;"><?php echo $text_enlarge; ?></span></td>
          <td style="padding-left: 15px; width: 296px; vertical-align: top;"><table width="100%">

Greatings

Xsecrets wrote: I just have to add in that the inline styles that he referenced are not the most offensive to me the fact that every exact same color gray box in the entire site is done with inline styles in the individual divs is what kills me so instead of changing one style .gray-box or something you have to edit 20-30 pages to change the color or the boxes.

Xsecrets wrote:

Qphoria wrote:Yep. I just tried it on TigerDirect.. changed my shipping address and poof tax was gone.
I find it retarded. But I guess it's correct.

It's because of the way the laws are written. The reason you don't pay tax on something ordered from another state is because federal law prohibits taxting interstate trade meaning the product is moved (shipped) from one state to another, so under the current laws you don't have to pay tax unless the company you are buying from has offices (or some sort of physical business presence) in the state where the product is delivered.

JNeuhoff wrote:

Qphoria wrote: - Tax Country/Zone is based on Shipping address. Should be based on Payment address. (Long time bug)

Isn't that the bug for which I posted a possible fix here?

Daniel wrote:can people just check if the mail problem is fixed?

I'm no longer using base64 encoding for the message parts.

Also just added the new version to the demo.

goldndelicious wrote:

Re: OpenCart v1.4.0
by twiggy » Mon Jan 11, 2010 11:43 am
Any one got a working install of this 1.4.0, I want to check it out but I'm in work so can't install it anywhere

i second that, stuck at work without anywhere to play. How about a screenshot of the front end?

Daniel wrote:i don't want full versions being sent. maybe just the modifed files.

I'm actually setting up mantis the bug tracker. it allows people to attach files and copy and paste code in.