PHP warning with ImportUser

shepo · November 5, 2021, 2:39pm

PHP notice: Undefined index: dob

not sure why I get this error

§

halftrainedharry · November 5, 2021, 3:51pm

As this is just a ‘notice’, you probably can simply ignore it (if the import works).

In case you want to examine it further:

Is there any information in the error message where (file, line number) this ‘error’ occurs?
What’s the PHP version on your server?
Do you have a column “dob” (date of birth) in your import file?

shepo · November 5, 2021, 4:06pm

Every line gets that error but it says import was successful.
PHP: PHP Version 7.4.23
No no such column

thanks

halftrainedharry · November 5, 2021, 4:47pm

I tried to reproduce the ‘error’ but can’t see it on my installation.

Just to be sure: We are talking about the extra User Import (as in your last thread).

I meant if the error message contains the line in the code where the ‘error’ occurs. Not the line in your import data.

shepo · November 5, 2021, 4:56pm

Can I ask what software do you use to create the clean csv

halftrainedharry · November 5, 2021, 5:03pm

I created a file by hand for testing purposes with just one user.

What columns do you have in your import file? Only the ones that are shown in the screenshot above?

I don’t think this error is necessarily a problem with your input file.

halftrainedharry · November 5, 2021, 5:24pm

I believe this line in the code is the problem:

github.com

gadgetto/UserImport/blob/0e6e2b319288561c31cab0463379096a9d44fc9f/core/components/userimport/model/userimport/importhandler.class.php#L496

    
      
                  return false;
              }
          
          
    // mobilephone
              if (!empty($fieldvalues['mobilephone']) && strlen($fieldvalues['mobilephone']) > 100) {
          		$this->modx->log(modX::LOG_LEVEL_WARN, '-> '.$this->modx->lexicon('userimport.import_users_row').$rowNumber.' '.$this->modx->lexicon('userimport.import_users_log_err_mobilephone_max_len'));
                  return false;
              }
          
          
    // dob -> needs check!
              if (!empty($fieldvalues['dob']) || $fieldvalues['dob'] == '0') {
                  // dob can be provided as UNIX timestamp or 
                  // any valid php date format
                  // -> but always saved and handled as UNIX timestamp!
                  if (!$this->validTimestamp($fieldvalues['dob']) && !$this->validDate($fieldvalues['dob'])) {
              		$this->modx->log(modX::LOG_LEVEL_WARN, '-> '.$this->modx->lexicon('userimport.import_users_row').$rowNumber.' '.$this->modx->lexicon('userimport.import_users_log_err_dob_invalid').$fieldvalues['dob']);
                      return false;
                  }
                  // if date format convert to timestamp
                  if ($this->validDate($fieldvalues['dob'])) {
                      $fieldvalues['dob'] = strtotime($fieldvalues['dob']);

Try changing it to this and see if the ‘error’ goes away.

if (isset($fieldvalues['dob']) && (!empty($fieldvalues['dob']) || $fieldvalues['dob'] == '0')) {

shepo · November 5, 2021, 5:28pm

Yes those were the only columns.
I was supplied in excel tried to use an excel csv (that wouldn’t import) - have mixed results with Sublime Text - but then they seem to import fine - from textedit - i just need a clear system - this dob thing is new since the recent update

halftrainedharry · November 5, 2021, 5:43pm

A csv file is just a text file so it shouldn’t make any difference which program you use.

Can you provide an example (with fake data obviously) that doesn’t import?