Duplicate code is one of the most pungent code smells. A rule that is often used is to re-structure code once it is duplicated in three or more places.
Common duplication problems, and corresponding solutions are:
Complex classes like EE_Import often do a lot of different things. To break such a class down, we need to identify a cohesive component within that class. A common approach to find such a component is to look for fields/methods that share the same prefixes, or suffixes. You can also have a look at the cohesion graph to spot any un-connected, or weakly-connected components.
Once you have determined the fields that belong together, you can apply the Extract Class refactoring. If the component makes sense as a sub-class, Extract Subclass is also a candidate, and is often faster.
While breaking up the class, it is a good idea to analyze how other classes use EE_Import, and based on these observations, apply Extract Interface, too.
| 1 | <?php if (!defined('EVENT_ESPRESSO_VERSION')) exit('No direct script access allowed'); | 
            ||
| 12 |  class EE_Import { | 
            ||
| 13 | |||
| 14 | const do_insert = 'insert';  | 
            ||
| 15 | const do_update = 'update';  | 
            ||
| 16 | const do_nothing = 'nothing';  | 
            ||
| 17 | |||
| 18 | |||
| 19 | // instance of the EE_Import object  | 
            ||
| 20 | private static $_instance = NULL;  | 
            ||
| 21 | |||
| 22 | private static $_csv_array = array();  | 
            ||
| 
                                                                                                    
                        
                         | 
                |||
| 23 | |||
| 24 | /**  | 
            ||
| 25 | *  | 
            ||
| 26 | * @var array of model names  | 
            ||
| 27 | */  | 
            ||
| 28 | private static $_model_list = array();  | 
            ||
| 29 | |||
| 30 | private static $_columns_to_save = array();  | 
            ||
| 31 | |||
| 32 | protected $_total_inserts = 0;  | 
            ||
| 33 | protected $_total_updates = 0;  | 
            ||
| 34 | protected $_total_insert_errors = 0;  | 
            ||
| 35 | protected $_total_update_errors = 0;  | 
            ||
| 36 | |||
| 37 | |||
| 38 | /**  | 
            ||
| 39 | * private constructor to prevent direct creation  | 
            ||
| 40 | * @Constructor  | 
            ||
| 41 | * @access private  | 
            ||
| 42 | * @return void  | 
            ||
| 43 | */  | 
            ||
| 44 |   private function __construct() { | 
            ||
| 45 | $this->_total_inserts = 0;  | 
            ||
| 46 | $this->_total_updates = 0;  | 
            ||
| 47 | $this->_total_insert_errors = 0;  | 
            ||
| 48 | $this->_total_update_errors = 0;  | 
            ||
| 49 | }  | 
            ||
| 50 | |||
| 51 | |||
| 52 | /**  | 
            ||
| 53 | * @ singleton method used to instantiate class object  | 
            ||
| 54 | * @ access public  | 
            ||
| 55 | * @return EE_Import  | 
            ||
| 56 | */  | 
            ||
| 57 | View Code Duplication | 	public static function instance() { | 
            |
| 58 | // check if class object is instantiated  | 
            ||
| 59 | 		if ( self::$_instance === NULL  or ! is_object( self::$_instance ) or ! ( self::$_instance instanceof EE_Import )) { | 
            ||
| 60 | self::$_instance = new self();  | 
            ||
| 61 | }  | 
            ||
| 62 | return self::$_instance;  | 
            ||
| 63 | }  | 
            ||
| 64 | |||
| 65 | /**  | 
            ||
| 66 | * Resets the importer  | 
            ||
| 67 | * @return EE_Import  | 
            ||
| 68 | */  | 
            ||
| 69 | 	public static function reset(){ | 
            ||
| 73 | |||
| 74 | |||
| 75 | |||
| 76 | |||
| 77 | /**  | 
            ||
| 78 | * @ generates HTML for a file upload input and form  | 
            ||
| 79 | * @ access public  | 
            ||
| 80 | * @param string $title - heading for the form  | 
            ||
| 81 | * @param string $intro - additional text explaing what to do  | 
            ||
| 82 | 	 * 	@param 	string 		$page - EE Admin page to direct form to - in the form "espresso_{pageslug}" | 
            ||
| 83 | * @param string $action - EE Admin page route array "action" that form will direct to  | 
            ||
| 84 | * @param string $type - type of file to import  | 
            ||
| 85 | * @ return string  | 
            ||
| 86 | */  | 
            ||
| 87 | 	public function upload_form ( $title, $intro, $form_url, $action, $type  ) { | 
            ||
| 88 | |||
| 89 | $form_url = EE_Admin_Page::add_query_args_and_nonce( array( 'action' => $action ), $form_url );  | 
            ||
| 90 | |||
| 91 | ob_start();  | 
            ||
| 92 | ?>  | 
            ||
| 93 | <div class="ee-upload-form-dv">  | 
            ||
| 94 | <h3><?php echo $title;?></h3>  | 
            ||
| 95 | <p><?php echo $intro;?></p>  | 
            ||
| 96 | |||
| 97 | <form action="<?php echo $form_url?>" method="post" enctype="multipart/form-data">  | 
            ||
| 98 | <input type="hidden" name="csv_submitted" value="TRUE" id="<?php echo time();?>">  | 
            ||
| 99 | <input name="import" type="hidden" value="<?php echo $type;?>" />  | 
            ||
| 100 | <input type="file" name="file[]" size="90" >  | 
            ||
| 101 | <input class="button-primary" type="submit" value="<?php _e( 'Upload File', 'event_espresso' );?>">  | 
            ||
| 102 | </form>  | 
            ||
| 103 | |||
| 104 | <p class="ee-attention">  | 
            ||
| 105 | <b><?php _e( 'Attention', 'event_espresso' );?></b><br/>  | 
            ||
| 106 | <?php echo sprintf( __( 'Accepts .%s file types only.', 'event_espresso' ), $type ) ;?>  | 
            ||
| 107 | <?php echo __( 'Please only import CSV files exported from Event Espresso, or compatible 3rd-party software.', 'event_espresso' );?>  | 
            ||
| 108 | </p>  | 
            ||
| 109 | |||
| 110 | </div>  | 
            ||
| 111 | |||
| 112 | <?php  | 
            ||
| 113 | $uploader = ob_get_clean();  | 
            ||
| 114 | return $uploader;  | 
            ||
| 115 | }  | 
            ||
| 116 | |||
| 117 | |||
| 118 | |||
| 119 | |||
| 120 | |||
| 121 | /**  | 
            ||
| 122 | * @Import Event Espresso data - some code "borrowed" from event espresso csv_import.php  | 
            ||
| 123 | * @access public  | 
            ||
| 124 | * @return boolean success  | 
            ||
| 125 | */  | 
            ||
| 126 | 	public function import() { | 
            ||
| 127 | |||
| 128 | require_once( EE_CLASSES . 'EE_CSV.class.php' );  | 
            ||
| 129 | $this->EE_CSV = EE_CSV::instance();  | 
            ||
| 130 | |||
| 131 | 		if ( isset( $_REQUEST['import'] )) { | 
            ||
| 132 | 			if( isset( $_POST['csv_submitted'] )) { | 
            ||
| 133 | |||
| 134 | 			    switch ( $_FILES['file']['error'][0] ) { | 
            ||
| 135 | case UPLOAD_ERR_OK:  | 
            ||
| 136 | $error_msg = FALSE;  | 
            ||
| 137 | break;  | 
            ||
| 138 | case UPLOAD_ERR_INI_SIZE:  | 
            ||
| 139 | 			            $error_msg = __("'The uploaded file exceeds the upload_max_filesize directive in php.ini.'", "event_espresso"); | 
            ||
| 140 | break;  | 
            ||
| 141 | case UPLOAD_ERR_FORM_SIZE:  | 
            ||
| 142 | 			            $error_msg = __('The uploaded file exceeds the MAX_FILE_SIZE directive that was specified in the HTML form.', "event_espresso"); | 
            ||
| 143 | break;  | 
            ||
| 144 | case UPLOAD_ERR_PARTIAL:  | 
            ||
| 145 | 			            $error_msg = __('The uploaded file was only partially uploaded.', "event_espresso"); | 
            ||
| 146 | break;  | 
            ||
| 147 | case UPLOAD_ERR_NO_FILE:  | 
            ||
| 148 | 			            $error_msg = __('No file was uploaded.', "event_espresso"); | 
            ||
| 149 | break;  | 
            ||
| 150 | case UPLOAD_ERR_NO_TMP_DIR:  | 
            ||
| 151 | 			            $error_msg = __('Missing a temporary folder.', "event_espresso"); | 
            ||
| 152 | break;  | 
            ||
| 153 | case UPLOAD_ERR_CANT_WRITE:  | 
            ||
| 154 | 			            $error_msg = __('Failed to write file to disk.', "event_espresso"); | 
            ||
| 155 | break;  | 
            ||
| 156 | case UPLOAD_ERR_EXTENSION:  | 
            ||
| 157 | 			            $error_msg = __('File upload stopped by extension.', "event_espresso"); | 
            ||
| 158 | break;  | 
            ||
| 159 | default:  | 
            ||
| 160 | 			            $error_msg = __('An unknown error occurred and the file could not be uploaded', "event_espresso"); | 
            ||
| 161 | break;  | 
            ||
| 162 | }  | 
            ||
| 163 | |||
| 164 | 				if ( ! $error_msg ) { | 
            ||
| 165 | |||
| 166 | $filename = $_FILES['file']['name'][0];  | 
            ||
| 167 | $file_ext = substr( strrchr( $filename, '.' ), 1 );  | 
            ||
| 168 | $file_type = $_FILES['file']['type'][0];  | 
            ||
| 169 | $temp_file = $_FILES['file']['tmp_name'][0];  | 
            ||
| 170 | $filesize = $_FILES['file']['size'][0] / 1024;//convert from bytes to KB  | 
            ||
| 171 | |||
| 172 | 					if ( $file_ext=='csv' ) { | 
            ||
| 173 | |||
| 174 | $max_upload = $this->EE_CSV->get_max_upload_size();//max upload size in KB  | 
            ||
| 175 | 						if ( $filesize < $max_upload || true) { | 
            ||
| 176 | |||
| 177 | $wp_upload_dir = str_replace( array( '\\', '/' ), DS, wp_upload_dir());  | 
            ||
| 178 | $path_to_file = $wp_upload_dir['basedir'] . DS . 'espresso' . DS . $filename;  | 
            ||
| 179 | |||
| 180 | 							if( move_uploaded_file( $temp_file, $path_to_file )) { | 
            ||
| 181 | |||
| 182 | // convert csv to array  | 
            ||
| 183 | $this->csv_array = $this->EE_CSV->import_csv_to_model_data_array( $path_to_file );  | 
            ||
| 184 | |||
| 185 | // was data successfully stored in an array?  | 
            ||
| 186 | 								if ( is_array( $this->csv_array ) ) { | 
            ||
| 187 | |||
| 188 | $import_what = str_replace( 'csv_import_', '', $_REQUEST['action'] );  | 
            ||
| 189 | $import_what = str_replace( '_', ' ', ucwords( $import_what ));  | 
            ||
| 190 | $processed_data = $this->csv_array;  | 
            ||
| 191 | $this->columns_to_save = FALSE;  | 
            ||
| 192 | |||
| 193 | // if any imports require funcky processing, we'll catch them in the switch  | 
            ||
| 194 | 									switch ($_REQUEST['action']) { | 
            ||
| 195 | |||
| 196 | case "import_events";  | 
            ||
| 197 | case "event_list";  | 
            ||
| 198 | $import_what = 'Event Details';  | 
            ||
| 199 | break;  | 
            ||
| 200 | |||
| 201 | case 'groupon_import_csv':  | 
            ||
| 202 | $import_what = 'Groupon Codes';  | 
            ||
| 203 | $processed_data = $this->process_groupon_codes();  | 
            ||
| 204 | break;  | 
            ||
| 205 | |||
| 206 | }  | 
            ||
| 207 | // save processed codes to db  | 
            ||
| 208 | 									if ( $this->save_csv_data_array_to_db( $processed_data, $this->columns_to_save ) ) { | 
            ||
| 209 | return TRUE;  | 
            ||
| 210 | |||
| 211 | }  | 
            ||
| 212 | 								} else { | 
            ||
| 213 | // no array? must be an error  | 
            ||
| 214 | 									EE_Error::add_error(sprintf(__("No file seems to have been uploaded", "event_espresso")), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 215 | return FALSE;  | 
            ||
| 216 | }  | 
            ||
| 217 | |||
| 218 | 							} else { | 
            ||
| 219 | 								EE_Error::add_error(sprintf(__("%s was not successfully uploaded", "event_espresso"),$filename), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 220 | return FALSE;  | 
            ||
| 221 | }  | 
            ||
| 222 | |||
| 223 | 						} else { | 
            ||
| 224 | 							EE_Error::add_error( sprintf(__("%s was too large of a file and could not be uploaded. The max filesize is %s' KB.", "event_espresso"),$filename,$max_upload), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 225 | return FALSE;  | 
            ||
| 226 | }  | 
            ||
| 227 | |||
| 228 | 					} else { | 
            ||
| 229 | 						EE_Error::add_error( sprintf(__("%s  had an invalid file extension, not uploaded", "event_espresso"),$filename), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 230 | return FALSE;  | 
            ||
| 231 | }  | 
            ||
| 232 | |||
| 233 | 				} else { | 
            ||
| 234 | EE_Error::add_error( $error_msg, __FILE__, __FUNCTION__, __LINE__ );  | 
            ||
| 235 | return FALSE;  | 
            ||
| 236 | }  | 
            ||
| 237 | |||
| 238 | }  | 
            ||
| 239 | }  | 
            ||
| 240 | return;  | 
            ||
| 241 | }  | 
            ||
| 242 | |||
| 243 | /**  | 
            ||
| 244 | * Given an array of data (usually from a CSV import) attempts to save that data to the db.  | 
            ||
| 245 | * If $model_name ISN'T provided, assumes that this is a 3d array, with toplevel keys being model names,  | 
            ||
| 246 | * next level being numeric indexes adn each value representing a model object, and the last layer down  | 
            ||
| 247 | * being keys of model fields and their proposed values.  | 
            ||
| 248 | * If $model_name IS provided, assumes a 2d array of the bottom two layers previously mentioned.  | 
            ||
| 249 | * If the CSV data says (in the metadata row) that it's from the SAME database,  | 
            ||
| 250 | * we treat the IDs in the CSV as the normal IDs, and try to update those records. However, if those  | 
            ||
| 251 | * IDs DON'T exist in the database, they're treated as temporary IDs,  | 
            ||
| 252 | * which can used elsewhere to refer to the same object. Once an item  | 
            ||
| 253 | * with a temporary ID gets inserted, we record its mapping from temporary  | 
            ||
| 254 | * ID to real ID, and use the real ID in place of the temporary ID  | 
            ||
| 255 | * when that temporary ID was used as a foreign key.  | 
            ||
| 256 | * If the CSV data says (in the metadata again) that it's from a DIFFERENT database,  | 
            ||
| 257 | * we treat all the IDs in the CSV as temporary ID- eg, if the CSV specifies an event with  | 
            ||
| 258 | * ID 1, and the database already has an event with ID 1, we assume that's just a coincidence,  | 
            ||
| 259 | * and insert a new event, and map it's temporary ID of 1 over to its new real ID.  | 
            ||
| 260 | * An important exception are non-auto-increment primary keys. If one entry in the  | 
            ||
| 261 | * CSV file has the same ID as one in the DB, we assume they are meant to be  | 
            ||
| 262 | * the same item, and instead update the item in the DB with that same ID.  | 
            ||
| 263 | * Also note, we remember the mappings permanently. So the 2nd, 3rd, and 10000th  | 
            ||
| 264 | * time you import a CSV from a different site, we remember their mappings, and  | 
            ||
| 265 | * will try to update the item in the DB instead of inserting another item (eg  | 
            ||
| 266 | * if we previously imported an event with temporary ID 1, and then it got a  | 
            ||
| 267 | * real ID of 123, we remember that. So the next time we import an event with  | 
            ||
| 268 | * temporary ID, from the same site, we know that it's real ID is 123, and will  | 
            ||
| 269 | * update that event, instead of adding a new event).  | 
            ||
| 270 | * @access public  | 
            ||
| 271 | * @param array $csv_data_array - the array containing the csv data produced from EE_CSV::import_csv_to_model_data_array()  | 
            ||
| 272 | * @param array $fields_to_save - an array containing the csv column names as keys with the corresponding db table fields they will be saved to  | 
            ||
| 273 | * @return TRUE on success, FALSE on fail  | 
            ||
| 274 | */  | 
            ||
| 275 | 	public function save_csv_data_array_to_db( $csv_data_array, $model_name = FALSE ) { | 
            ||
| 276 | |||
| 277 | |||
| 278 | $success = FALSE;  | 
            ||
| 279 | $error = FALSE;  | 
            ||
| 280 | //whther to treat this import as if it's data froma different database or not  | 
            ||
| 281 | //ie, if it IS from a different database, ignore foreign keys whihf  | 
            ||
| 282 | $export_from_site_a_to_b = true;  | 
            ||
| 283 | // first level of array is not table information but a table name was passed to the function  | 
            ||
| 284 | // array is only two levels deep, so let's fix that by adding a level, else the next steps will fail  | 
            ||
| 285 | 		if($model_name){ | 
            ||
| 286 | $csv_data_array = array($csv_data_array);  | 
            ||
| 287 | }  | 
            ||
| 288 | // begin looking through the $csv_data_array, expecting the toplevel key to be the model's name...  | 
            ||
| 289 | $old_site_url = 'none-specified';  | 
            ||
| 290 | |||
| 291 | //hanlde metadata  | 
            ||
| 292 | 		if(isset($csv_data_array[EE_CSV::metadata_header]) ){ | 
            ||
| 293 | $csv_metadata = array_shift($csv_data_array[EE_CSV::metadata_header]);  | 
            ||
| 294 | //ok so its metadata, dont try to save it to ehte db obviously...  | 
            ||
| 295 | 			if(isset($csv_metadata['site_url']) && $csv_metadata['site_url'] == site_url()){ | 
            ||
| 296 | 				EE_Error::add_attention(sprintf(__("CSV Data appears to be from the same database, so attempting to update data", "event_espresso"))); | 
            ||
| 297 | $export_from_site_a_to_b = false;  | 
            ||
| 298 | 			}else{ | 
            ||
| 299 | $old_site_url = isset( $csv_metadata['site_url']) ? $csv_metadata['site_url'] : $old_site_url;  | 
            ||
| 300 | 				EE_Error::add_attention(sprintf(__("CSV Data appears to be from a different database (%s instead of %s), so we assume IDs in the CSV data DO NOT correspond to IDs in this database", "event_espresso"),$old_site_url,site_url())); | 
            ||
| 301 | };  | 
            ||
| 302 | unset($csv_data_array[EE_CSV::metadata_header]);  | 
            ||
| 303 | }  | 
            ||
| 304 | /**  | 
            ||
| 305 | * @var $old_db_to_new_db_mapping 2d array: toplevel keys being model names, bottom-level keys being the original key, and  | 
            ||
| 306 | * the value will be the newly-inserted ID.  | 
            ||
| 307 | * If we have already imported data from the same website via CSV, it shoudl be kept in this wp option  | 
            ||
| 308 | */  | 
            ||
| 309 | 	   $old_db_to_new_db_mapping = get_option('ee_id_mapping_from'.sanitize_title($old_site_url),array()); | 
            ||
| 310 | 	   if( $old_db_to_new_db_mapping){ | 
            ||
| 311 | 		   EE_Error::add_attention(sprintf(__("We noticed you have imported data via CSV from %s before. Because of this, IDs in your CSV have been mapped to their new IDs in %s", "event_espresso"),$old_site_url,site_url())); | 
            ||
| 312 | }  | 
            ||
| 313 | $old_db_to_new_db_mapping = $this->save_data_rows_to_db($csv_data_array, $export_from_site_a_to_b, $old_db_to_new_db_mapping);  | 
            ||
| 314 | |||
| 315 | //save the mapping from old db to new db in case they try re-importing the same data from the same website again  | 
            ||
| 316 | 		update_option('ee_id_mapping_from'.sanitize_title($old_site_url),$old_db_to_new_db_mapping); | 
            ||
| 317 | |||
| 318 | View Code Duplication | 		if ( $this->_total_updates > 0 ) { | 
            |
| 319 | 			EE_Error::add_success( sprintf(__("%s existing records in the database were updated.", "event_espresso"),$this->_total_updates)); | 
            ||
| 320 | $success = true;  | 
            ||
| 321 | }  | 
            ||
| 322 | View Code Duplication | 		if ( $this->_total_inserts > 0 ) { | 
            |
| 323 | 			EE_Error::add_success(sprintf(__("%s new records were added to the database.", "event_espresso"),$this->_total_inserts)); | 
            ||
| 324 | $success = true;  | 
            ||
| 325 | }  | 
            ||
| 326 | |||
| 327 | View Code Duplication | 		if ( $this->_total_update_errors > 0 ) { | 
            |
| 328 | 			EE_Error::add_error(sprintf(__("'One or more errors occurred, and a total of %s existing records in the database were <strong>not</strong> updated.'", "event_espresso"),$this->_total_update_errors), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 329 | $error = true;  | 
            ||
| 330 | }  | 
            ||
| 331 | View Code Duplication | 		if ( $this->_total_insert_errors > 0 ) { | 
            |
| 332 | 			EE_Error::add_error(sprintf(__("One or more errors occurred, and a total of %s new records were <strong>not</strong> added to the database.'", "event_espresso"),$this->_total_insert_errors), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 333 | $error = true;  | 
            ||
| 334 | }  | 
            ||
| 335 | |||
| 336 | //lastly, we need to update the datetime and ticket sold amounts  | 
            ||
| 337 | //as those may ahve been affected by this  | 
            ||
| 338 | EEM_Datetime::instance()->update_sold( EEM_Datetime::instance()->get_all() );  | 
            ||
| 339 | EEM_Ticket::instance()->update_tickets_sold(EEM_Ticket::instance()->get_all());  | 
            ||
| 340 | |||
| 341 | // if there was at least one success and absolutely no errors  | 
            ||
| 342 | 		if ( $success && ! $error ) { | 
            ||
| 343 | return TRUE;  | 
            ||
| 344 | 		} else { | 
            ||
| 345 | return FALSE;  | 
            ||
| 346 | }  | 
            ||
| 347 | |||
| 348 | }  | 
            ||
| 349 | |||
| 350 | |||
| 351 | /**  | 
            ||
| 352 | * Processes the array of data, given the knowledge that it's from the same database or a different one,  | 
            ||
| 353 | * and the mapping from temporary IDs to real IDs.  | 
            ||
| 354 | * If the data is from a different database, we treat the primary keys and their corresponding  | 
            ||
| 355 | * foreign keys as "temp Ids", basically identifiers that get mapped to real primary keys  | 
            ||
| 356 | * in the real target database. As items are inserted, their temporary primary keys  | 
            ||
| 357 | * are mapped to the real IDs in the target database. Also, before doing any update or  | 
            ||
| 358 | * insert, we replace all the temp ID which are foreign keys with their mapped real IDs.  | 
            ||
| 359 | * An exception: string primary keys are treated as real IDs, or else we'd need to  | 
            ||
| 360 | * dynamically generate new string primary keys which would be very awkard for the country table etc.  | 
            ||
| 361 | * Also, models with no primary key are strange too. We combine use their primar key INDEX (a  | 
            ||
| 362 | * combination of fields) to create a unique string identifying the row and store  | 
            ||
| 363 | * those in the mapping.  | 
            ||
| 364 | *  | 
            ||
| 365 | * If the data is from the same database, we usually treat primary keys as real IDs.  | 
            ||
| 366 | * An exception is if there is nothing in the database for that ID. If that's the case,  | 
            ||
| 367 | * we need to insert a new row for that ID, and then map from the non-existent ID  | 
            ||
| 368 | * to the newly-inserted real ID.  | 
            ||
| 369 | * @param type $csv_data_array  | 
            ||
| 370 | * @param type $export_from_site_a_to_b  | 
            ||
| 371 | * @param type $old_db_to_new_db_mapping  | 
            ||
| 372 | * @return array updated $old_db_to_new_db_mapping  | 
            ||
| 373 | */  | 
            ||
| 374 | 	public function save_data_rows_to_db( $csv_data_array, $export_from_site_a_to_b, $old_db_to_new_db_mapping ) { | 
            ||
| 375 | 		foreach ( $csv_data_array as $model_name_in_csv_data => $model_data_from_import ) { | 
            ||
| 376 | //now check that assumption was correct. If  | 
            ||
| 377 | View Code Duplication | 			if ( EE_Registry::instance()->is_model_name($model_name_in_csv_data)) { | 
            |
| 378 | $model_name = $model_name_in_csv_data;  | 
            ||
| 379 | 			}else { | 
            ||
| 380 | // no table info in the array and no table name passed to the function?? FAIL  | 
            ||
| 381 | 				EE_Error::add_error( __('No table information was specified and/or found, therefore the import could not be completed','event_espresso'), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 382 | return FALSE;  | 
            ||
| 383 | }  | 
            ||
| 384 | /* @var $model EEM_Base */  | 
            ||
| 385 | $model = EE_Registry::instance()->load_model($model_name);  | 
            ||
| 386 | |||
| 387 | //so without further ado, scanning all the data provided for primary keys and their inital values  | 
            ||
| 388 | 			foreach ( $model_data_from_import as $model_object_data ) { | 
            ||
| 389 | //before we do ANYTHING, make sure the csv row wasn't just completely blank  | 
            ||
| 390 | $row_is_completely_empty = true;  | 
            ||
| 391 | 				foreach($model_object_data as $field){ | 
            ||
| 392 | 					if($field){ | 
            ||
| 393 | $row_is_completely_empty = false;  | 
            ||
| 394 | }  | 
            ||
| 395 | }  | 
            ||
| 396 | 				if($row_is_completely_empty){ | 
            ||
| 397 | continue;  | 
            ||
| 398 | }  | 
            ||
| 399 | //find the PK in the row of data (or a combined key if  | 
            ||
| 400 | //there is no primary key)  | 
            ||
| 401 | 				if($model->has_primary_key_field()){ | 
            ||
| 402 | $id_in_csv = $model_object_data[$model->primary_key_name()];  | 
            ||
| 403 | 				}else{ | 
            ||
| 404 | $id_in_csv = $model->get_index_primary_key_string($model_object_data);  | 
            ||
| 405 | }  | 
            ||
| 406 | |||
| 407 | |||
| 408 | $model_object_data = $this->_replace_temp_ids_with_mappings( $model_object_data, $model, $old_db_to_new_db_mapping, $export_from_site_a_to_b );  | 
            ||
| 409 | //now we need to decide if we're going to add a new model object given the $model_object_data,  | 
            ||
| 410 | //or just update.  | 
            ||
| 411 | 				if($export_from_site_a_to_b){ | 
            ||
| 412 | $what_to_do = $this->_decide_whether_to_insert_or_update_given_data_from_other_db( $id_in_csv, $model_object_data, $model, $old_db_to_new_db_mapping );  | 
            ||
| 413 | 				}else{//this is just a re-import | 
            ||
| 414 | $what_to_do = $this->_decide_whether_to_insert_or_update_given_data_from_same_db( $id_in_csv, $model_object_data, $model, $old_db_to_new_db_mapping );  | 
            ||
| 415 | }  | 
            ||
| 416 | 				if( $what_to_do == self::do_nothing ) { | 
            ||
| 417 | continue;  | 
            ||
| 418 | }  | 
            ||
| 419 | |||
| 420 | //double-check we actually want to insert, if that's what we're planning  | 
            ||
| 421 | //based on whether this item would be unique in the DB or not  | 
            ||
| 422 | 				if( $what_to_do == self::do_insert ) { | 
            ||
| 423 | //we're supposed to be inserting. But wait, will this thing  | 
            ||
| 424 | //be acceptable if inserted?  | 
            ||
| 425 | $conflicting = $model->get_one_conflicting( $model_object_data, false );  | 
            ||
| 426 | 					if($conflicting){ | 
            ||
| 427 | //ok, this item would conflict if inserted. Just update the item that it conflicts with.  | 
            ||
| 428 | $what_to_do = self::do_update;  | 
            ||
| 429 | //and if this model has a primary key, remember its mapping  | 
            ||
| 430 | 						if($model->has_primary_key_field()){ | 
            ||
| 431 | $old_db_to_new_db_mapping[$model_name][$id_in_csv] = $conflicting->ID();  | 
            ||
| 432 | $model_object_data[$model->primary_key_name()] = $conflicting->ID();  | 
            ||
| 433 | 						}else{ | 
            ||
| 434 | //we want to update this conflicting item, instead of inserting a conflicting item  | 
            ||
| 435 | //so we need to make sure they match entirely (its possible that they only conflicted on one field, but we need them to match on other fields  | 
            ||
| 436 | //for the WHERE conditions in the update). At the time of this comment, there were no models like this  | 
            ||
| 437 | 							foreach($model->get_combined_primary_key_fields() as $key_field){ | 
            ||
| 438 | $model_object_data[$key_field->get_name()] = $conflicting->get($key_field->get_name());  | 
            ||
| 439 | }  | 
            ||
| 440 | }  | 
            ||
| 441 | }  | 
            ||
| 442 | }  | 
            ||
| 443 | 				if( $what_to_do == self::do_insert ) { | 
            ||
| 444 | $old_db_to_new_db_mapping = $this->_insert_from_data_array( $id_in_csv, $model_object_data, $model, $old_db_to_new_db_mapping );  | 
            ||
| 445 | 				}elseif( $what_to_do == self::do_update ) { | 
            ||
| 446 | $old_db_to_new_db_mapping = $this->_update_from_data_array( $id_in_csv, $model_object_data, $model, $old_db_to_new_db_mapping );  | 
            ||
| 447 | 				}else{ | 
            ||
| 448 | throw new EE_Error( sprintf( __( 'Programming error. We shoudl be inserting or updating, but instead we are being told to "%s", whifh is invalid', 'event_espresso' ), $what_to_do ) );  | 
            ||
| 449 | }  | 
            ||
| 450 | }  | 
            ||
| 451 | }  | 
            ||
| 452 | return $old_db_to_new_db_mapping;  | 
            ||
| 453 | }  | 
            ||
| 454 | |||
| 455 | |||
| 456 | |||
| 457 | /**  | 
            ||
| 458 | * Decides whether or not to insert, given that this data is from another database.  | 
            ||
| 459 | * So, if the primary key of this $model_object_data already exists in the database,  | 
            ||
| 460 | * it's just a coincidence and we should still insert. The only time we should  | 
            ||
| 461 | * update is when we know what it maps to, or there's something that would  | 
            ||
| 462 | * conflict (and we should instead just update that conflicting thing)  | 
            ||
| 463 | * @param string $id_in_csv  | 
            ||
| 464 | * @param array $model_object_data by reference so it can be modified  | 
            ||
| 465 | * @param EEM_Base $model  | 
            ||
| 466 | * @param array $old_db_to_new_db_mapping by reference so it can be modified  | 
            ||
| 467 | * @return string one of the consts on this class that starts with do_*  | 
            ||
| 468 | */  | 
            ||
| 469 | 	protected function _decide_whether_to_insert_or_update_given_data_from_other_db( $id_in_csv, $model_object_data, $model, $old_db_to_new_db_mapping ) { | 
            ||
| 470 | $model_name = $model->get_this_model_name();  | 
            ||
| 471 | //if it's a site-to-site export-and-import, see if this modelobject's id  | 
            ||
| 472 | //in the old data that we know of  | 
            ||
| 473 | 		if( isset($old_db_to_new_db_mapping[$model_name][$id_in_csv]) ){ | 
            ||
| 474 | return self::do_update;  | 
            ||
| 475 | 		}else{ | 
            ||
| 476 | return self::do_insert;  | 
            ||
| 477 | }  | 
            ||
| 478 | }  | 
            ||
| 479 | |||
| 480 | /**  | 
            ||
| 481 | * If this thing basically already exists in the database, we want to update it;  | 
            ||
| 482 | * otherwise insert it (ie, someone tweaked the CSV file, or the item was  | 
            ||
| 483 | * deleted in the database so it should be re-inserted)  | 
            ||
| 484 | * @param type $id_in_csv  | 
            ||
| 485 | * @param type $model_object_data  | 
            ||
| 486 | * @param EEM_Base $model  | 
            ||
| 487 | * @param type $old_db_to_new_db_mapping  | 
            ||
| 488 | * @return  | 
            ||
| 489 | */  | 
            ||
| 490 | 	protected function _decide_whether_to_insert_or_update_given_data_from_same_db( $id_in_csv, $model_object_data, $model ) { | 
            ||
| 491 | //in this case, check if this thing ACTUALLY exists in the database  | 
            ||
| 492 | 		if( $model->get_one_conflicting( $model_object_data ) ){ | 
            ||
| 493 | return self::do_update;  | 
            ||
| 494 | 		}else{ | 
            ||
| 495 | return self::do_insert;  | 
            ||
| 496 | }  | 
            ||
| 497 | }  | 
            ||
| 498 | |||
| 499 | /**  | 
            ||
| 500 | * Using the $old_db_to_new_db_mapping array, replaces all the temporary IDs  | 
            ||
| 501 | * with their mapped real IDs. Eg, if importing from site A to B, the mapping  | 
            ||
| 502 | * file may indicate that the ID "my_event_id" maps to an actual event ID of 123.  | 
            ||
| 503 | * So this function searches for any event temp Ids called "my_event_id" and  | 
            ||
| 504 | * replaces them with 123.  | 
            ||
| 505 | * Also, if there is no temp ID for the INT foreign keys from another database,  | 
            ||
| 506 | * replaces them with 0 or the field's default.  | 
            ||
| 507 | * @param type $model_object_data  | 
            ||
| 508 | * @param EEM_Base $model  | 
            ||
| 509 | * @param type $old_db_to_new_db_mapping  | 
            ||
| 510 | * @param boolean $export_from_site_a_to_b  | 
            ||
| 511 | * @return array updated model object data with temp IDs removed  | 
            ||
| 512 | */  | 
            ||
| 513 | 	protected function _replace_temp_ids_with_mappings( $model_object_data, $model, $old_db_to_new_db_mapping, $export_from_site_a_to_b ) { | 
            ||
| 514 | //if this model object's primary key is in the mapping, replace it  | 
            ||
| 515 | if( $model->has_primary_key_field() &&  | 
            ||
| 516 | $model->get_primary_key_field()->is_auto_increment() &&  | 
            ||
| 517 | isset( $old_db_to_new_db_mapping[ $model->get_this_model_name() ] ) &&  | 
            ||
| 518 | 				isset( $old_db_to_new_db_mapping[ $model->get_this_model_name() ][ $model_object_data[ $model->primary_key_name() ] ] ) ) { | 
            ||
| 519 | $model_object_data[ $model->primary_key_name() ] = $old_db_to_new_db_mapping[ $model->get_this_model_name() ][ $model_object_data[ $model->primary_key_name() ] ];  | 
            ||
| 520 | }  | 
            ||
| 521 | |||
| 522 | 		try{ | 
            ||
| 523 | $model_name_field = $model->get_field_containing_related_model_name();  | 
            ||
| 524 | $models_pointed_to_by_model_name_field = $model_name_field->get_model_names_pointed_to();  | 
            ||
| 525 | 		}catch( EE_Error $e ){ | 
            ||
| 526 | $model_name_field = NULL;  | 
            ||
| 527 | $models_pointed_to_by_model_name_field = array();  | 
            ||
| 528 | }  | 
            ||
| 529 | 		foreach( $model->field_settings( true )  as $field_obj ){ | 
            ||
| 530 | 			if( $field_obj instanceof EE_Foreign_Key_Int_Field ) { | 
            ||
| 531 | $models_pointed_to = $field_obj->get_model_names_pointed_to();  | 
            ||
| 532 | $found_a_mapping = false;  | 
            ||
| 533 | 				foreach( $models_pointed_to as $model_pointed_to_by_fk ) { | 
            ||
| 534 | |||
| 535 | 					if( $model_name_field ){ | 
            ||
| 536 | $value_of_model_name_field = $model_object_data[ $model_name_field->get_name() ];  | 
            ||
| 537 | View Code Duplication | 						if( $value_of_model_name_field == $model_pointed_to_by_fk ) { | 
            |
| 538 | $model_object_data[ $field_obj->get_name() ] = $this->_find_mapping_in(  | 
            ||
| 539 | $model_object_data[ $field_obj->get_name() ],  | 
            ||
| 540 | $model_pointed_to_by_fk,  | 
            ||
| 541 | $old_db_to_new_db_mapping,  | 
            ||
| 542 | $export_from_site_a_to_b );  | 
            ||
| 543 | $found_a_mapping = true;  | 
            ||
| 544 | break;  | 
            ||
| 545 | }  | 
            ||
| 546 | View Code Duplication | 					}else{ | 
            |
| 547 | $model_object_data[ $field_obj->get_name() ] = $this->_find_mapping_in(  | 
            ||
| 548 | $model_object_data[ $field_obj->get_name() ],  | 
            ||
| 549 | $model_pointed_to_by_fk,  | 
            ||
| 550 | $old_db_to_new_db_mapping,  | 
            ||
| 551 | $export_from_site_a_to_b );  | 
            ||
| 552 | $found_a_mapping = true;  | 
            ||
| 553 | }  | 
            ||
| 554 | //once we've found a mapping for this field no need to continue  | 
            ||
| 555 | 					if( $found_a_mapping ) { | 
            ||
| 556 | break;  | 
            ||
| 557 | }  | 
            ||
| 558 | |||
| 559 | |||
| 560 | }  | 
            ||
| 561 | 			}else{ | 
            ||
| 562 | //it's a string foreign key (which we leave alone, because those are things  | 
            ||
| 563 | //like country names, which we'd really rather not make 2 USAs etc (we'd actually  | 
            ||
| 564 | //prefer to just update one)  | 
            ||
| 565 | //or it's just a regular value that ought to be replaced  | 
            ||
| 566 | }  | 
            ||
| 567 | }  | 
            ||
| 568 | //  | 
            ||
| 569 | 		if( $model instanceof EEM_Term_Taxonomy ){ | 
            ||
| 570 | $model_object_data = $this->_handle_split_term_ids( $model_object_data );  | 
            ||
| 571 | }  | 
            ||
| 572 | return $model_object_data;  | 
            ||
| 573 | }  | 
            ||
| 574 | |||
| 575 | /**  | 
            ||
| 576 | * If the data was exported PRE-4.2, but then imported POST-4.2, then the term_id  | 
            ||
| 577 | * this term-taxonomy refers to may be out-of-date so we need to update it.  | 
            ||
| 578 | * see https://make.wordpress.org/core/2015/02/16/taxonomy-term-splitting-in-4-2-a-developer-guide/  | 
            ||
| 579 | * @param type $model_object_data  | 
            ||
| 580 | * @return array new model object data  | 
            ||
| 581 | */  | 
            ||
| 582 | 	protected function _handle_split_term_ids( $model_object_data ){ | 
            ||
| 583 | 		if( isset( $model_object_data['term_id'] ) && isset( $model_object_data[ 'taxonomy' ]) && apply_filters( 'FHEE__EE_Import__handle_split_term_ids__function_exists', function_exists( 'wp_get_split_term' ), $model_object_data ) ) { | 
            ||
| 584 | $new_term_id = wp_get_split_term( $model_object_data[ 'term_id' ], $model_object_data[ 'taxonomy' ] );  | 
            ||
| 585 | 			if( $new_term_id ){ | 
            ||
| 586 | $model_object_data[ 'term_id' ] = $new_term_id;  | 
            ||
| 587 | }  | 
            ||
| 588 | }  | 
            ||
| 589 | return $model_object_data;  | 
            ||
| 590 | }  | 
            ||
| 591 | /**  | 
            ||
| 592 | * Given the object's ID and its model's name, find it int he mapping data,  | 
            ||
| 593 | * bearing in mind where it came from  | 
            ||
| 594 | * @param type $object_id  | 
            ||
| 595 | * @param string $model_name  | 
            ||
| 596 | * @param array $old_db_to_new_db_mapping  | 
            ||
| 597 | * @param type $export_from_site_a_to_b  | 
            ||
| 598 | * @return int  | 
            ||
| 599 | */  | 
            ||
| 600 | 	protected function _find_mapping_in( $object_id, $model_name, $old_db_to_new_db_mapping, $export_from_site_a_to_b) { | 
            ||
| 601 | 		if(	isset( $old_db_to_new_db_mapping[ $model_name ][ $object_id ] ) ){ | 
            ||
| 602 | |||
| 603 | return $old_db_to_new_db_mapping[ $model_name ][ $object_id ];  | 
            ||
| 604 | 			}elseif( $object_id == '0' || $object_id == '' ) { | 
            ||
| 605 | //leave as-is  | 
            ||
| 606 | return $object_id;  | 
            ||
| 607 | 			}elseif( $export_from_site_a_to_b ){ | 
            ||
| 608 | //we couldn't find a mapping for this, and it's from a different site,  | 
            ||
| 609 | //so blank it out  | 
            ||
| 610 | return NULL;  | 
            ||
| 611 | 			}elseif( ! $export_from_site_a_to_b ) { | 
            ||
| 612 | //we coudln't find a mapping for this, but it's from thsi DB anyway  | 
            ||
| 613 | //so let's just leave it as-is  | 
            ||
| 614 | return $object_id;  | 
            ||
| 615 | }  | 
            ||
| 616 | }  | 
            ||
| 617 | |||
| 618 | /**  | 
            ||
| 619 | *  | 
            ||
| 620 | * @param type $id_in_csv  | 
            ||
| 621 | * @param type $model_object_data  | 
            ||
| 622 | * @param EEM_Base $model  | 
            ||
| 623 | * @param type $old_db_to_new_db_mapping  | 
            ||
| 624 | * @return array updated $old_db_to_new_db_mapping  | 
            ||
| 625 | */  | 
            ||
| 626 | 	protected function _insert_from_data_array( $id_in_csv, $model_object_data, $model, $old_db_to_new_db_mapping ) { | 
            ||
| 627 | //remove the primary key, if there is one (we don't want it for inserts OR updates)  | 
            ||
| 628 | //we'll put it back in if we need it  | 
            ||
| 629 | 		if($model->has_primary_key_field() && $model->get_primary_key_field()->is_auto_increment()){ | 
            ||
| 630 | $effective_id = $model_object_data[$model->primary_key_name()];  | 
            ||
| 631 | unset($model_object_data[$model->primary_key_name()]);  | 
            ||
| 632 | 		}else{ | 
            ||
| 633 | $effective_id = $model->get_index_primary_key_string( $model_object_data );  | 
            ||
| 634 | }  | 
            ||
| 635 | //the model takes care of validating the CSV's input  | 
            ||
| 636 | 		try{ | 
            ||
| 637 | $new_id = $model->insert($model_object_data);  | 
            ||
| 638 | 			if( $new_id ){ | 
            ||
| 639 | $old_db_to_new_db_mapping[$model->get_this_model_name()][$id_in_csv] = $new_id;  | 
            ||
| 640 | $this->_total_inserts++;  | 
            ||
| 641 | 				EE_Error::add_success( sprintf(__("Successfully added new %s (with id %s) with csv data %s", "event_espresso"),$model->get_this_model_name(),$new_id, implode(",",$model_object_data))); | 
            ||
| 642 | 			}else{ | 
            ||
| 643 | $this->_total_insert_errors++;  | 
            ||
| 644 | //put the ID used back in there for the error message  | 
            ||
| 645 | 				if($model->has_primary_key_field()){ | 
            ||
| 646 | $model_object_data[$model->primary_key_name()] = $effective_id;  | 
            ||
| 647 | }  | 
            ||
| 648 | 				EE_Error::add_error( sprintf(__("Could not insert new %s with the csv data: %s", "event_espresso"),$model->get_this_model_name(),http_build_query($model_object_data)), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 649 | }  | 
            ||
| 650 | 		}catch(EE_Error $e){ | 
            ||
| 651 | $this->_total_insert_errors++;  | 
            ||
| 652 | 			if($model->has_primary_key_field()){ | 
            ||
| 653 | $model_object_data[$model->primary_key_name()] = $effective_id;  | 
            ||
| 654 | }  | 
            ||
| 655 | 			EE_Error::add_error( sprintf(__("Could not insert new %s with the csv data: %s because %s", "event_espresso"),$model->get_this_model_name(),implode(",",$model_object_data),$e->getMessage()), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 656 | }  | 
            ||
| 657 | return $old_db_to_new_db_mapping;  | 
            ||
| 658 | }  | 
            ||
| 659 | |||
| 660 | /**  | 
            ||
| 661 | * Given the model object data, finds the row to update and updates it  | 
            ||
| 662 | * @param string|int $id_in_csv  | 
            ||
| 663 | * @param array $model_object_data  | 
            ||
| 664 | * @param EEM_Base $model  | 
            ||
| 665 | * @param array $old_db_to_new_db_mapping  | 
            ||
| 666 | * @return array updated $old_db_to_new_db_mapping  | 
            ||
| 667 | */  | 
            ||
| 668 | 	protected function _update_from_data_array( $id_in_csv,  $model_object_data, $model, $old_db_to_new_db_mapping ) { | 
            ||
| 669 | 		try{ | 
            ||
| 670 | //let's keep two copies of the model object data:  | 
            ||
| 671 | //one for performing an update, one for everthing else  | 
            ||
| 672 | $model_object_data_for_update = $model_object_data;  | 
            ||
| 673 | 			if($model->has_primary_key_field()){ | 
            ||
| 674 | $conditions = array($model->primary_key_name() => $model_object_data[$model->primary_key_name()]);  | 
            ||
| 675 | //remove the primary key because we shouldn't use it for updating  | 
            ||
| 676 | unset($model_object_data_for_update[$model->primary_key_name()]);  | 
            ||
| 677 | 			}elseif($model->get_combined_primary_key_fields() > 1 ){ | 
            ||
| 678 | $conditions = array();  | 
            ||
| 679 | 				foreach($model->get_combined_primary_key_fields() as $key_field){ | 
            ||
| 680 | $conditions[$key_field->get_name()] = $model_object_data[$key_field->get_name()];  | 
            ||
| 681 | }  | 
            ||
| 682 | 			}else{ | 
            ||
| 683 | $model->primary_key_name();//this shoudl just throw an exception, explaining that we dont have a primary key (or a combine dkey)  | 
            ||
| 684 | }  | 
            ||
| 685 | |||
| 686 | $success = $model->update($model_object_data_for_update,array($conditions));  | 
            ||
| 687 | 			if($success){ | 
            ||
| 688 | $this->_total_updates++;  | 
            ||
| 689 | 				EE_Error::add_success( sprintf(__("Successfully updated %s with csv data %s", "event_espresso"),$model->get_this_model_name(),implode(",",$model_object_data_for_update))); | 
            ||
| 690 | //we should still record the mapping even though it was an update  | 
            ||
| 691 | //because if we were going to insert somethign but it was going to conflict  | 
            ||
| 692 | //we would have last-minute decided to update. So we'd like to know what we updated  | 
            ||
| 693 | //and so we record what record ended up being updated using the mapping  | 
            ||
| 694 | 				if( $model->has_primary_key_field() ){ | 
            ||
| 695 | $new_key_for_mapping = $model_object_data[ $model->primary_key_name() ];  | 
            ||
| 696 | 				}else{ | 
            ||
| 697 | //no primary key just a combined key  | 
            ||
| 698 | $new_key_for_mapping = $model->get_index_primary_key_string( $model_object_data );  | 
            ||
| 699 | }  | 
            ||
| 700 | $old_db_to_new_db_mapping[ $model->get_this_model_name() ][ $id_in_csv ] = $new_key_for_mapping;  | 
            ||
| 701 | 			}else{ | 
            ||
| 702 | $matched_items = $model->get_all(array($conditions));  | 
            ||
| 703 | 				if( ! $matched_items){ | 
            ||
| 704 | //no items were matched (so we shouldn't have updated)... but then we should have inserted? what the heck?  | 
            ||
| 705 | $this->_total_update_errors++;  | 
            ||
| 706 | 					EE_Error::add_error( sprintf(__("Could not update %s with the csv data: '%s' for an unknown reason (using WHERE conditions %s)", "event_espresso"),$model->get_this_model_name(),http_build_query($model_object_data),http_build_query($conditions)), __FILE__, __FUNCTION__, __LINE__ ); | 
            ||
| 707 | 				}else{ | 
            ||
| 708 | $this->_total_updates++;  | 
            ||
| 709 | 					EE_Error::add_success( sprintf(__("%s with csv data '%s' was found in the database and didn't need updating because all the data is identical.", "event_espresso"),$model->get_this_model_name(),implode(",",$model_object_data))); | 
            ||
| 710 | }  | 
            ||
| 711 | }  | 
            ||
| 712 | 		}catch(EE_Error $e){ | 
            ||
| 713 | $this->_total_update_errors++;  | 
            ||
| 714 | 			$basic_message = sprintf(__("Could not update %s with the csv data: %s because %s", "event_espresso"),$model->get_this_model_name(),implode(",",$model_object_data),$e->getMessage()); | 
            ||
| 715 | $debug_message = $basic_message . ' Stack trace: ' . $e->getTraceAsString();  | 
            ||
| 716 | EE_Error::add_error( "$basic_message | $debug_message", __FILE__, __FUNCTION__, __LINE__ );  | 
            ||
| 717 | }  | 
            ||
| 718 | return $old_db_to_new_db_mapping;  | 
            ||
| 719 | }  | 
            ||
| 720 | |||
| 721 | /**  | 
            ||
| 722 | * Gets the number of inserts performed since importer was instantiated or reset  | 
            ||
| 723 | * @return int  | 
            ||
| 724 | */  | 
            ||
| 725 | 	public function get_total_inserts(){ | 
            ||
| 728 | /**  | 
            ||
| 729 | * Gets the number of insert errors since importer was instantiated or reset  | 
            ||
| 730 | * @return int  | 
            ||
| 731 | */  | 
            ||
| 732 | 	public function get_total_insert_errors(){ | 
            ||
| 735 | /**  | 
            ||
| 736 | * Gets the number of updates performed since importer was instantiated or reset  | 
            ||
| 737 | * @return int  | 
            ||
| 738 | */  | 
            ||
| 739 | 	public function get_total_updates(){ | 
            ||
| 742 | /**  | 
            ||
| 743 | * Gets the number of update errors since importer was instantiated or reset  | 
            ||
| 744 | * @return int  | 
            ||
| 745 | */  | 
            ||
| 746 | 	public function get_total_update_errors(){ | 
            ||
| 749 | |||
| 750 | |||
| 751 | |||
| 752 | |||
| 753 | |||
| 754 | }  | 
            ||
| 755 | /* End of file EE_Import.class.php */  | 
            ||
| 756 | /* Location: /includes/classes/EE_Import.class.php */  | 
            ||
| 757 | ?>  | 
            
This check marks private properties in classes that are never used. Those properties can be removed.