Trying to access array offset on value of type bool. Fixed.

[fa-stable.git] / includes / ui / contacts_view.inc

diff --git a/includes/ui/contacts_view.inc b/includes/ui/contacts_view.inc
index bd25e2820e5003bfbbf51b3bf8f8dd8710c3d3f3..23474adc9e32aca2d66b90f06189569d24a13f87 100644 (file)
--- a/includes/ui/contacts_view.inc
+++ b/includes/ui/contacts_view.inc
@@ -21,11 +21,11 @@ class contacts extends simple_crud {
        var $sub_class;
        var $class;
        
-       function contacts($name, $id, $class, $subclass=null) {
+       function __construct($name, $id, $class, $subclass=null) {
                $fields = array('ref','name','name2','address', 'phone', 'phone2','fax',
                        'email','lang','notes', 'assgn' =>array('fld'=>'contacts'));
 
-               $this->simple_crud($name, $fields);
+               parent::__construct($name, $fields);
                $this->class = $class;
                $this->subclass = $subclass;
                $this->entity = $id;
@@ -35,13 +35,11 @@ class contacts extends simple_crud {
                br();
 
                        $contacts = get_crm_persons($this->class, $this->subclass, $this->entity);
-//                             check_value('show_inactive'));
                        start_table(TABLESTYLE, "width='80%'");
 
                        $th = array(_('Assignment'), _("Reference"), _("Full Name"), _("Phone"), _("Sec Phone"), _("Fax"),
                                _("email"),  '',
                                " ");
-               //      inactive_control_column($th);
 
                        table_header($th);
                        $k = 0; 
@@ -50,7 +48,7 @@ class contacts extends simple_crud {
                        while ($last)
                        {
                                        
-                               if ($myrow['id'] != $last['id']) {
+                               if (@$myrow['id'] != $last['id']) {
                                        alt_table_row_color($k);
                                        label_cell(implode('<br>',$description));
                                        label_cell($last["ref"]);
@@ -59,11 +57,10 @@ class contacts extends simple_crud {
                                        label_cell($last["phone2"]);
                                        label_cell($last["fax"]);
                                        email_cell($last["email"]);
-               //      inactive_control_cell($myrow["id"], $myrow["inactive"], 'crm_contacts', 'id');
 
                                        edit_button_cell("{$this->name}Edit[{$last['id']}]", _("Edit"));
 
-                       // delete button removes only category unless this is the last contact for this person
+                                       // delete button removes only category unless this is the last contact for this person
                                        delete_button_cell("{$this->name}Delete[{$last['id']}]", _("Delete"));
                                        end_row();
                                        $description = array();
@@ -76,7 +73,7 @@ class contacts extends simple_crud {
                                }
                        }
 
-//inactive_control_row($th);
+                       //inactive_control_row($th);
                        end_table(1);
        }
        
@@ -137,7 +134,7 @@ class contacts extends simple_crud {
                        set_focus('ref');
                        return false;
                }
-               if (count(@$_POST['assgn']) == 0) {
+               if (count_array(@$_POST['assgn']) == 0) {
                        display_error(_("You have to select at least one category."));
                        set_focus('assgn');
                        return false;