CRM-2274 CiviMember Import- behaves differently than CiviCRM imports

    Details

    • Type: Bug
    • Status: Done/Fixed
    • Priority: Minor
    • Resolution: Fixed/Completed
    • Affects Version/s: 1.8
    • Fix Version/s: 2.0
    • Component/s: CiviMember
    • Labels:
      None

      Description

      When importing into CiviMember, I would expect similar behavior as what happens in CiviCRM imports, especially since the interface is similar.

      I've noticed two minor differences:

      1) Validation testing before step 3: At step 3, CiviMember tells you the number of valid rows in the file. It doesn't look like CiviMember actually does the validation testing. (Though, I noticed, if I click the browser back button from step 4, I see the output that I would expect to see with the "rows with errors" count)

      In my test file, Membership Type and some emails are not valid, and they make it through step 3 just fine.

      2) The Import_Errors file: When CiviMember reports Import_Errors in a csv file, and the original csv file has column headers, the Import_Errors file does not increment the record number to account for the column header row. This makes it difficult to quickly find entries by the excel row number (although it may be more accurate, arguably). CiviCRM Import_Errors file does the increment in these situations.

      You can see both of the above using my test file for individuals, with the saved matching fields called testbug2.

        Attachments

          Activity

          [CRM-2274] CiviMember Import- behaves differently than CiviCRM imports
          Rupam Jaiswal added a comment -

          fixed in rev 11566.

          Manish Zope added a comment -

          Verify for 2.0

          Junia Biswas added a comment -
          • if the csv file has 4 record with invalid data and 4 valid data, while importing it shows 4 rows imported sucessfully but none of record imported in the database.
          Manish Zope added a comment -

          Issue is already verified for 2.0, hence closing it.

          Kiran Jagtap added a comment -

          Tested and Confirmed. ( r - 16177 )

            People

            • Assignee:
              Kiran Jagtap
              Reporter:
              AnnieTM

              Dates

              • Created:
                Updated:
                Resolved: