[#MDL-16520] Failed portfolio exports result in an ugly error message that users should never see

Issue Details (XML | Word | Printable)

Key:	MDL-16520
Type:	Sub-task
Status:	Open
Priority:	Minor
Assignee:	Penny Leach
Reporter:	Martin Dougiamas
Votes:	0
Watchers:	1

Operations

Add/Edit UI Mockup to this issue

If you were logged in you would be able to see more operations.

Moodle

MDL-14591

Failed portfolio exports result in an ugly error message that users should never see

Created: 16/Sep/08 11:12 AM Updated: 01/Mar/09 08:05 PM

Return to search

Component/s:

Portfolio API

Affects Version/s:

2.0

Fix Version/s:

2.0

File Attachments:

None

Image Attachments:

1. add_warning.png
(12 kB)

2. The error message that users should never see.jpg
(65 kB)

Participants:	Dan Poltawski, Martin Dougiamas and Penny Leach
Security Level:	None
QA Assignee:	Martin Dougiamas
Affected Branches:	MOODLE_20_STABLE
Fixed Branches:	MOODLE_20_STABLE

Description

« Hide

1) Start an export from a forum post to Box.net
2) Change your mind when authenticating
3) Go back to the forum discussion
4) Observe the carnage (error message contains CONTROLS and also prevents forum display)

I think it should still show the forum discussion, and perhaps have a little alerting "!" icon or something next to the "Add to portfolio" link to show that it's in progress and some , clicking on it would go to a separate screen that tells you what is going on and lets you cancel/complete etc.

All

Comments

Change History

Version Control

Sort Order:

mjollnir_ committed 4 files to 'Moodle CVS' - 16/Sep/08 06:41 PM

MDL-16520 - make the already exporting portfolio error more user friendly

MODIFY	lib/portfoliolib.php	Rev. 1.61	(+36 -4 lines)
MODIFY	pix/t/portfoliono.gif	Rev. 1.1	(+0 -0 lines)
ADD	portfolio/Attic/already.php	Rev. 1.1	(+0 -0 lines)
MODIFY	lang/en_utf8/portfolio.php	Rev. 1.28	(+4 -1 lines)

mjollnir_ committed 3 files to 'Moodle CVS' - 16/Sep/08 08:28 PM

MDL-16520 - print info about current export

MODIFY	portfolio/Attic/already.php	Rev. 1.2	(+32 -0 lines)
MODIFY	lib/portfoliolib.php	Rev. 1.62	(+8 -5 lines)
MODIFY	lang/en_utf8/portfolio.php	Rev. 1.29	(+0 -1 lines)

Mitsuhiro Yoshida committed 6 files to 'Lang CVS' - 17/Sep/08 12:49 AM

MDL-16520 - Translated new strings for portfolio.
MDL-16483 - Translated new strings for simple test.
MDL-16519 - Translated a new string for box.net porfolio.
MDL-16414 - Translated a new string for download portfolio.
~~MDL-15402~~ - Translated a new string for repository.

MODIFY	ja_utf8/repository.php	Rev. 1.26	(+2 -1 lines)
MODIFY	ja_utf8/portfolio.php	Rev. 1.24	(+4 -2 lines)
MODIFY	ja_utf8/portfolio_boxnet.php	Rev. 1.9	(+2 -1 lines)
MODIFY	ja_utf8/simpletest.php	Rev. 1.9	(+6 -2 lines)
MODIFY	ja_utf8/portfolio_download.php	Rev. 1.3	(+2 -1 lines)
MODIFY	ja_utf8/README	Rev. 1.815	(+1 -1 lines)