Skip to content

Instantly share code, notes, and snippets.

@SchumacherFM
Created May 6, 2015 08:12
  • Star 0 You must be signed in to star a gist
  • Fork 0 You must be signed in to fork a gist
Star You must be signed in to star a gist
Save SchumacherFM/378dc6d18ed4b3a8cb37 to your computer and use it in GitHub Desktop.
Magento1: How not to load models/collections in a grid renderer. This shows a very bad example 😞. Do not use it or πŸ”«.
<?php
/*
@SchumacherFM: The render() method is executed in each row
*/
/**
* Product: Vendor_GridActions
* ID:
* File: app/code/local/Vendor/GridActions/Block/Adminhtml/Sales/Order/Grid/Widget/Renderer/Trackingnumber.php
*/
class Vendor_GridActions_Block_Adminhtml_Sales_Order_Grid_Widget_Renderer_Trackingnumber extends Mage_Adminhtml_Block_Widget_Grid_Column_Renderer_Select
{
protected $_column = false;
public function render(Varien_Object $row)
{
if ($this->_column !== false) {
$column = $this->_column;
} else {
$column = $this->getColumn();
}
#$colId = $column->getName() ? $column->getName() : $column->getId();
$colId = 'tracking-input';
$orderId = $row->getEntityId();
$order = Mage::getModel('sales/order')->load($orderId);
if (!$order->getId()) {
return '';
}
$html = '';
if ($order->canShip()) {
$html = '<input name="' . $colId . '-' . $row->getId() . '" rel="' . $row->getId() . '" class="input-text ' . $colId . '"
value="' . $row->getData($column->getIndex()) . '" style="width:97%;" onclick="vendorOnClickJs(this)"/>';
} else if (!$order->canShip() && $order->getStatus() !== Mage_Sales_Model_Order::STATE_CANCELED && $order->getStatus() !== Mage_Sales_Model_Order::STATE_CLOSED) {
$trackingNumbers = array();
if (Mage::helper('vendor/utils')->mageVersionCompare(Mage::getVersion(), '1.4.0.0', '>=')) {
$trackingUrl = Mage::helper('shipping')->getTrackingPopupUrlBySalesModel($order);
} else {
$trackingUrl = Mage::helper('shipping')->getTrackingPopUpUrlByOrderId($order->getEntityId());
}
// Starting from Magento 1.6, the trackingnumber field has been renamed from number to track_number
if (Mage::helper('vendor/utils')->mageVersionCompare(Mage::getVersion(), '1.6.0.0', '>=')) {
$tracks = Mage::getModel('sales/order_shipment_track')
->getCollection()
->addAttributeToSelect('track_number')
->setOrderFilter($row->getEntityId());
foreach ($tracks as $track) {
$trackingNumbers[] = '<a href="#" onclick="popWin(\'' . $trackingUrl . '\',\'trackorder\',\'width=800,height=600,left=0,top=0,resizable=yes,scrollbars=yes\')" >' . $this->escapeHtml($track->getTrackNumber()) . '</a>';
}
} else {
$tracks = Mage::getModel('sales/order_shipment_track')
->getCollection()
->addAttributeToSelect('number')
->setOrderFilter($row->getEntityId());
foreach ($tracks as $track) {
$trackingNumbers[] = '<a href="#" onclick="popWin(\'' . $trackingUrl . '\',\'trackorder\',\'width=800,height=600,left=0,top=0,resizable=yes,scrollbars=yes\')" >' . $this->escapeHtml($track->getNumber()) . '</a>';
}
}
$html = implode(', ', $trackingNumbers);
if (Mage::getStoreConfigFlag('gridactions/general/add_trackingnumber_from_grid_shipped')) {
if (count($tracks) > 0) {
$html .= '<br/>';
}
$html .= '<input name="' . $colId . '-' . $row->getId() . '" rel="' . $row->getId() . '" class="input-text ' . $colId . '"
value="' . $row->getData($column->getIndex()) . '" style="width:97%;" onclick="vendorOnClickJs(this)"/>';
}
}
return $html;
}
public function renderCombined($row, $column)
{
$this->_column = $column;
return $this->render($row);
}
/*
* Return dummy filter.
*/
public function getFilter()
{
return false;
}
/* Fix for compatibility with Magento version <1.4 */
public function escapeHtml($data, $allowedTags = null)
{
if (Mage::helper('vendor/utils')->mageVersionCompare(Mage::getVersion(), '1.4.0.0', '>=')) {
return Mage::helper('core')->escapeHtml($data, $allowedTags);
} else {
return Mage::helper('core')->htmlEscape($data, $allowedTags);
}
}
}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment