← Back to team overview

mahara-contributors team mailing list archive

[Bug 1897199] Re: importing comments: wrong name to display and content

 

Instructions for testing:

- Need to checkout the patch before starting to work since I made
changes on the export and the import as well.

- The attached DB:
    The db contains 6 users:
    admin
    user00001
    user00002
    user00003
    user00004
    user00005

    all passwords are 'mahara'

    The only one that has portfolios id user00001
    It has a page PageA that contains 
    A. 4 comments:
     - a public comment and a private comment from user00002
     - a public comment and a private comment from user00003
    B. an assessment block that contains:
      - a public assessment and a draft assessment from user00002
      - a public assessment and a draft assessment from user00003
    It also has a collection CollectionA that contains PageA


- Using this DB, there three steps to follow to test the peer assessment import and the comments import

    * Export: create the Leap2A export files that will be used to test the import
    - Login as admin
    - go to admin/users/bulkexport.php
    - select Leap2A as the export format, set user00001 as username to export
    - click Export accounts
    - save the 'Leap2A account' export

    - login as user00001
    - go to Main menu -> Manage -> Export
    - select 'All my data'
    - Generate export
    - save the 'all my data' export

    - click 'Continue'
    - select 'Just some of my pages'
    - select PageA
    - Generate export
    - save the 'PageA' export

    - click 'Continue'
    - select 'Just some of my collections'
    - select CollectionA
    - Generate export
    - save the 'CollectionA' export


    * Remove user00001 and user00002 from the site

    - login as admin
    - go to Admin menu-> People -> People search
    - click on user00001
    - suspend or delete account
    - delete account
    - go to Admin menu-> People -> People search
    - click on user00002
    - suspend or delete account
    - delete account


    * Import: test the leap2A files created on first export step
    
    - login as user00004
    - go to Main menu -> Manage -> Import
    - Browse and select 'pageA export'
    - Check all data is fine in PageA

    - go to Main menu -> Manage -> Import
    - Browse and select 'pageA export'
    - Check all data is fine in PageA inside CollectionA

    - login as admin
    - Admin menu -> People -> Add person
    - on 'Creation method' check 'Upload Leap2A file' and browse for 'All my data' export
    - set username= user00001 password= qweQWE123!@# and create account
    - login as user00001
    - check pageA from collectionA has correct information

    - login as admin and delete user00001
    - go to admin/users/bulkimport.php
    - set 'Bulk export file' to the 'account export' location
    if the file it's in tmp folder and it's called 'account_export.zip' write
    /tmp/account_export.zip
    - click 'Import'
    - login as user00001 and check PageA has correct information


- To check if PageA has correct information
    For peer assessment 
    - there should be only 2 peer assessments. These are the public ones. The draft should not be displayed
    - the authors of the peer assessments should not be displayed and they should not link to any existing account. There should be a "Imported peer assessment" text instead
    For comments
    - all comments should be imported and displayed to the oner of the imported portfolio
    - the private comments should not be displayed to any other than the owner
    - the author names of the comments should not be displayed and they should not link to any existing account. There should be a "Imported feedback" text instead


** Attachment added: "import_peerassessment_comment.pg"
   https://bugs.launchpad.net/mahara/+bug/1897199/+attachment/5415505/+files/import_peerassessment_comment.pg

** Description changed:

  When exporting from a site and importing to a different site, there is
  no consent of the author of a comment in this other site to display
  their name. We should be replacing the name for something like 'Imported
  feedback'.
  
  The comment contents are not display correctly. The text is replaced by
  the string "Comment" like is shown in the attached image.
  
- 
  Cases to take into consideration when testing:
  
  Comment export + import testing
  
  Export
-  1. Full account
-  2. Single page
-  3. Collection (should be the same)
-  4. Bulk
+  1. Full account
+  2. Single page
+  3. Collection (should be the same)
+  4. Bulk
  
  Comments needed:
-  1. Public on page
-  2. Private on page
-  3. Public on artefact, i.e. journal entry so we don't need site data
-  4. Private on artefact
- 
+  1. Public on page
+  2. Private on page
+  3. Public on artefact, i.e. journal entry so we don't need site data
+  4. Private on artefact
  
  Import
-  1. Interactive, i.e. own account
-  2. Admin menu ->People->Add person
-  3. Bulk
- 
- - Set up a DB that has scenarios and dump it for easier testing
- - See peer assessment config testing for testing matrix example
+  1. Interactive, i.e. own account
+  2. Admin menu ->People->Add person
+  3. Bulk

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1897199

Title:
  importing comments: wrong name to display and content

Status in Mahara:
  In Progress

Bug description:
  When exporting from a site and importing to a different site, there is
  no consent of the author of a comment in this other site to display
  their name. We should be replacing the name for something like
  'Imported feedback'.

  IMPORTANT: Since the .xml text can be easily edited, it's best not to
  check if the username already exists on the system. So no names or ids
  are exported or imported for comments or peer assessments.

  The comment contents are not display correctly. The text is replaced
  by the string "Comment" like is shown in the attached image.

  Cases to take into consideration when testing:

  Comment export + import testing

  Export
   1. Full account
   2. Single page
   3. Collection (should be the same)
   4. Bulk

  Comments needed:
   1. Public on page
   2. Private on page
   3. Public on artefact, i.e. journal entry so we don't need site data
   4. Private on artefact

  Import
   1. Interactive, i.e. own account
   2. Admin menu ->People->Add person
   3. Bulk

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1897199/+subscriptions


References