• 0 Vote(s) - 0 Average
  • 1
  • 2
  • 3
  • 4
  • 5
Require Urgent Help

#1
[eluser]greedyh4mster[/eluser]
Hi!

I am trying to create a notification object that serves to remind my users about certain messages. I am not sure whether am I heading towards the correct direction or not. Would seriously appreciate some help from you guys.

However I have encountered several problems along the way, and really hope that you guys could help me out with it. Assuming that my coding design is correct, the major problem that I have encountered is:

Code:
Fatal error: Call to undefined method stdClass::make_status_links()

What I am trying to achieve is to create a notification class, and a collection of notifications class. So when it is time to grab all the notifications for a specific user, I am able to call the notifications class, collection of notification object, and manipulate the data.

Below are the codes:

This is a test function from the user controller:

Code:
function test(){
      $this->load->library('notifications');

      $notifications = $this->notifications->get_notifications_active(1);
   }

This is the controller class for the notification object:
Code:
<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');

class Notification {

   private $ci;
   private $error = '';
  
   function __construct(){
      $this->ci =& get_instance();
      $this->ci->load->model('notification_model');
   }
  
   function add($user_id = '', $message_index = '', $action_path = ''){
      $this->ci->notification_model->user_id = $user_id;
      $this->ci->notification_model->message_index = $message_index;
      $this->ci->notification_model->action_path = $action_path;
      
      return $this->ci->notification_model->add();
   }

   function dismiss($notification_id = ''){
       $this->ci->notification_model->notification_id = $notification_id;

       if($this->ci->notification_model->dismiss()){
          return TRUE;
       } else{
          $this->error = 'notification_id_invalid';
          return FALSE;
       }
   }

   function remindmelater($notification_id = ''){
      $this->ci->notification_model->notification_id = $notification_id;

      if($this->ci->notification_model->remindmelater()){
          return TRUE;
       } else{
          $this->error = 'notification_id_invalid';
          return FALSE;
       }
   }

   function make_links($notification_id = ''){
      $this->ci->notification_model->notification_id = $notification_id;
      
      var_dump($this->ci->notification_model->make_status_links());
      
      return $this->ci->notification_model->make_status_links();
   }

   function error(){
      return $this->ci->lang->line($this->error);
   }
  
}

?>

This is the model class for the notification object:
Code:
<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');

class Notification_model extends Model {

   public $notification_id;
   public $user_id;
   public $message_index;
   public $action_path;
   public $status;
   public $time;

   private $links = array();

   function __construct(){
      parent::Model();
      $this->load->config('fr3b_config');
   }

   function add(){
      $this->status = $this->config->item('notification_status_active');
      $this->time = date("Y-m-d H:i:s", time());
      
      $this->db->insert('notifications', $this);
      
      return $this->db->insert_id();
   }

   function dismiss(){
      $this->status = $this->config->item('notification_status_dismiss');
      
      return $this->_set_status();
   }

   function remindmelater(){
      $this->status = $this->config->item('notification_status_remindmelater');

      return $this->_set_status();
   }

   function _set_status(){
      $this->db->update(
         'notifications',
         array('status' => $this->status),
         array('notification_id' => $this->notification_id)
      );

      return $this->db->affected_rows();
   }

   function make_status_links(){
      $this->links['dismiss'] = anchor('account/notifications/'.$this->config->item('notification_status_dismiss').'/'.$this->notification_id, 'Dismiss');
      $this->links['remindmelater'] = anchor('account/notifications/'.$this->config->item('notification_status_remindmelater').'/'.$this->notification_id, 'Remind me later');
  
      return $this->links;
   }
  
}
?>

This is the class for a collection of notification objects:
Code:
<?php if ( ! defined('BASEPATH')) exit('No direct script access allowed');

class Notifications extends Notification {

   private $ci;

   private $notifications = array();

   function __construct(){
      $this->ci =& get_instance();
   }

   function get_notifications(){

      //Get all notifications regardless of status

   }

   function get_notifications_active($user_id = ''){
      $this->ci->load->config('fr3b_config');
      
      //Get all notifications that has status of 'active'
      $this->ci->db->where('status', $this->ci->config->item('notification_status_active'));
      $this->ci->db->where('user_id', $user_id);
      $this->notifications = $this->ci->db->get('notifications')->result();

      foreach($this->notifications as $notification){
         $notification->links = $this->make_links($notification->notification_id);
      }

      return $this->notifications;
   }

}

?>

#2
[eluser]frist44[/eluser]
in the construct function of the Notifications class, you need to load the parent construct because that's where the model is loaded.

Code:
function __construct(){
      parent::__construct();
      $this->ci =& get_instance();
   }

#3
[eluser]frist44[/eluser]
Also, it's usually best practice to do all your data access to the database in models. So this function would ideally be moved over to the model: get_notifications_active()


Digg   Delicious   Reddit   Facebook   Twitter   StumbleUpon  


Users browsing this thread:
1 Guest(s)


  Theme © 2014 iAndrew  
Powered By MyBB, © 2002-2019 MyBB Group.