Jump to content
Sign in to follow this  
sinco

Помогите доработать историю заказа

Recommended Posts

Необходимо добавить в историю заказа колонку с именем пользователя, который добавил строчку, вот скрин желаемого:

post-673798-0-43329300-1407510491_thumb.jpg

Что бы в момент записи имя пользователя, который что либо добавляет в историю записывалось в таблицу oc_order_history в отдельную колонку.
Думаю вопрос актуален для многих, так как есть очень много магазинов где работает много пользователей!
 

 

post-673798-0-43329300-1407510491_thumb.jpg

Share this post


Link to post
Share on other sites

Вам нужна платная помощь или рассказ, как это сделать?

Share this post


Link to post
Share on other sites

Вам нужна платная помощь или рассказ, как это сделать?

он скорее разделом ошибся, т.к. 

 

 

Раздел для помощи разработчикам. Если вы столкнулись с проблемами во время создания своего дополнения или изменения кода движка - вам сюда.

Если Вы НЕ разработчик - пишите вопросы в другие разделы.

За готовыми решениями обращаться сюда бессмысленно.

Share this post


Link to post
Share on other sites

Так делов-то рассказать

 

Добавить в таблицу order_history поле user_id

 

Изменить в модели метод  addOrderHistory

добавив поле

В контроллере передать  в метод user_id

 

В шаблоне добавить нужный блок( ячейку, строку )

 

В контроллере передать нужное в шаблон

В модели в методе getOrderHistories

 

связать user_id с именем пользователя.

Share this post


Link to post
Share on other sites

Зачем user_id и затем всякие джойны для вытаскивания имени, если кроме имени больше ничего не надо?

Надо сразу в ордер-хистори хранить имя пользователя.

Уйдёт масса лишних дёрганий.

И добавится возможность безболезненно увольнять пользователей-менеджеров, удаляя их записи из базы без потери истории (а кто это сделал) и без блокировок или висяков.

Share this post


Link to post
Share on other sites

Реализовал это немного другим способом!

Вместе с новой строкой, добавляется еще одна, в которой прописано:

Polzovatel "admin" (#1) dobavil noviy status

Реализовал это все следующим способом:

файл admin/model/sale/order.php

1. После

public function getTotalEmailsByProductsOrdered($products) {
        $implode = array();
        
        foreach ($products as $product_id) {
            $implode[] = "op.product_id = '" . $product_id . "'";
        }
                
        $query = $this->db->query("SELECT COUNT(DISTINCT email) AS total FROM `" . DB_PREFIX . "order` o LEFT JOIN " . DB_PREFIX . "order_product op ON (o.order_id = op.order_id) WHERE (" . implode(" OR ", $implode) . ") AND o.order_status_id <> '0'");
        
        return $query->row['total'];
    }

добавил

public function addOrderHistory2($order_id, $data) {
        $userId = $this->user->getId();
        $userName = $this->user->getUserName();
        $comment = "User "".$userName."" (#".$userId.") made some changes to this order.";
        
        $this->db->query("INSERT INTO `".DB_PREFIX."order_history` SET `order_id` = '".(int)$order_id."', `order_status_id` = '".(int)$data['order_status_id']."', `notify` = '0', `comment` = '".$this->db->escape($comment)."', `date_added` = NOW()");
    }

2.

public function addOrderHistory($order_id, $data) {

заменил на

public function addOrderHistory($order_id, $data) {
        
        /* [w] */
        $this->addOrderHistory2($order_id, $data);
        /* [w] */

3.

public function editOrder($order_id, $data) {

заменил на

public function editOrder($order_id, $data) {
        
        /* [w] */
        $this->addOrderHistory2($order_id, $data);
        /* [w] */

Share this post


Link to post
Share on other sites

Зачем user_id и затем всякие джойны для вытаскивания имени, если кроме имени больше ничего не надо?

Надо сразу в ордер-хистори хранить имя пользователя.

Уйдёт масса лишних дёрганий.

И добавится возможность безболезненно увольнять пользователей-менеджеров, удаляя их записи из базы без потери истории (а кто это сделал) и без блокировок или висяков.

Если имена пользователей не пересекаются.

 

 

 

Удалять пользователей  - плохая практика.

Share this post


Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
You are posting as a guest. If you have an account, please sign in.
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Sign in to follow this  

  • Recently Browsing   0 members

    No registered users viewing this page.

×

Important Information

On our site, cookies are used and personal data is processed to improve the user interface. To find out what and what personal data we are processing, please go to the link. If you click "I agree," it means that you understand and accept all the conditions specified in this Privacy Notice.