Credit goes to Eloy Lafuente.

Also, cheating, and committing a whitespace fix in the same commit.

Conflicts:

	question/engine/questionusage.php

AMOS BEGIN
 MOV [totalpointsx,mod_quiz],[totalmarksx,mod_quiz]
AMOS END

Searches dataroot/pix/ and dataroot/pix_plugins/type/plugin for icons
to override base install

This is unlikely to be the long term solution and so MDL-32329 has been
created to address that

Removed the context parameter for blackboard and blackboard_six formats.

This parameter was removed in 4d188926

Also fixed a comment typo I spotted whilst reviewing

I've tested that unit with debug developer, debug notices and
debug none and it passes always. So, as far as I cannot find
any reason to have those expectError() there... they are out now.

(b60d2174 is where
all the portfolio testing stuff was deleted)