[#MDL-16403] Portfolio: create a folder generate a notice

Issue Details (XML | Word | Printable)

Key:	MDL-16403
Type:	Bug
Status:	Resolved
Resolution:	Fixed
Priority:	Minor
Assignee:	Dongsheng Cai
Reporter:	Jerome Mouneyrac
Votes:	0
Watchers:	0

Operations

Add/Edit UI Mockup to this issue

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

Moodle

Portfolio: create a folder generate a notice

Created: 09/Sep/08 02:55 PM Updated: 17/Sep/08 04:15 AM

Return to search

Component/s:

Portfolio API

Affects Version/s:

2.0

Fix Version/s:

2.0

File Attachments:

None

Image Attachments:

1. Screenshot-Exporting to portfolio - Mozilla Firefox.png
(96 kB)

Environment:

LAMP

Issue Links:

Cloners

This issue has a clone:

~~MDL-16539~~ Warning when exporting to boxlib and creating a new folder

Participants:	Dongsheng Cai, Jerome Mouneyrac and Penny Leach
Security Level:	None
Resolved date:	10/Sep/08
Affected Branches:	MOODLE_20_STABLE
Fixed Branches:	MOODLE_20_STABLE

Description

« Hide

Forum post: Add to portoflio
Select box.net
Create a new folder
A notice is displayed (see screen capture)

All

Comments

Change History

Version Control

Sort Order:

mjollnir_ committed 1 file to 'Moodle CVS' - 09/Sep/08 10:08 PM

~~MDL-16403~~A - save state in portfolio before redirecting to steal control url.

MODIFY

lib/portfolio/exporter.php

Rev. 1.3

(+3 -1 lines)

[ Permalink | « Hide ]

Penny Leach added a comment - 09/Sep/08 10:09 PM

Fixed. This was caused by the boxnet changing state (saving the auth token) but the api not saving the changes before redirecting to the steal control url.

good catch.

Penny Leach made changes - 09/Sep/08 10:09 PM

Field	Original Value	New Value
Fix Version/s		2.0 [ 10122 ]
Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

[ Permalink | « Hide ]

Penny Leach added a comment - 09/Sep/08 10:18 PM

Sorry, previous comment related to ~~MDL-16402~~ , not this bug.

Penny Leach made changes - 09/Sep/08 10:18 PM

Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]

[ Permalink | « Hide ]

Penny Leach added a comment - 09/Sep/08 10:19 PM

Reassigning to DS. This notice is in the boxnet client he wrote, not the portfolio code.

(Also it still needs to move to lib!)

Penny Leach made changes - 09/Sep/08 10:19 PM

Assignee

Penny Leach [ mjollnir ]

Dongsheng Cai [ dongsheng ]

Dongsheng Cai committed 4 files to 'Moodle CVS' - 10/Sep/08 11:10 AM

~~MDL-16403~~
1. move boxnet lib to moodle lib folder
2. fix a notice

DEL	repository/boxnet/Attic/boxlibphp5.php	Rev. 1.21	(+0 -0 lines)
MODIFY	repository/boxnet/repository.class.php	Rev. 1.41	(+5 -2 lines)
ADD	lib/boxlib.php	Rev. 1.1	(+0 -0 lines)
MODIFY	portfolio/type/boxnet/Attic/lib.php	Rev. 1.17	(+1 -1 lines)

[ Permalink | « Hide ]

Dongsheng Cai added a comment - 10/Sep/08 11:11 AM

Fixed, and moved boxnet lib to moodle lib folder

Dongsheng Cai made changes - 10/Sep/08 11:11 AM

Status	Reopened [ 4 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Penny Leach made changes - 17/Sep/08 04:15 AM

Link

This issue has a clone ~~MDL-16539~~ [ ~~MDL-16539~~ ]

All content on this web site is made available under the GNU General Public License, unless otherwise stated.