mahara-contributors team mailing list archive
-
mahara-contributors team
-
Mailing list archive
-
Message #60326
[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