1) We use to put only one release for each branch both in the "affects versions" and "fix versions". So I guess the correct ones for this should be:

affects versions: 1.9, 2.0 (HEAD)
fix versions: 1.9.1, 2.0 (HEAD)

But, generally, for something fixed in a branch (1.9) we don't use to include HEAD (2.0). It's supposed that we always merge changes there, so finally I'd leave this as:

affects versions: 1.9
fix versions: 1.9.1

And will use the 2.0 for things coming exclusively in 2.0.

Just an explanation. Ciao

P.S.: Of course, once you have resolved one bug you should mark it as resolved (fixed) to allow it to be reviewed on weekly reviews.

• error(get_string("invalidpubkey", 'mnet') . $errmsg ,'peers.php?step=update&hostid='.$mnet_peer->id);
  + //error(get_string("invalidpubkey", 'mnet') . $errmsg ,'peers.php?step=update&hostid='.$mnet_peer->id);
  + print_error("invalidpubkey", 'mnet', 'peers.php?step=update&hostid='.$mnet_peer->id, $errmsg);

2/

• error(get_string('noguestpost', 'forum').
• ' You are not allowed to make modifications to this RSS feed at this time.',
• $referrer);

+ //print_error(get_string('noguestpost', 'forum').
+ // ' You are not allowed to make modifications to this RSS feed at this time.',
+ // $referrer);
+ print_error('noguestpost', 'forum', $referrer, 'You are not allowed to make modifications to this RSS feed at this time.');

}

3/

• error(get_string('noguestpost', 'forum').
• ' You are not allowed to make modifications to this RSS feed at this time.',
• $referrer);

+ //print_error(get_string('noguestpost', 'forum').
+ // ' You are not allowed to make modifications to this RSS feed at this time.',
+ // $referrer);
+ print_error('noguestpost', 'forum', $referrer, 'You are not allowed to make modifications to this RSS feed at this time.');

4/

• error(get_string('missingfield', 'error', $name). " ".
• get_string('erroronline', 'error', $linenum) .". ".
• get_string('processingstops', 'error'),
• 'uploaduser.php?sesskey='.$USER->sesskey);

+ //error(get_string('missingfield', 'error', $name). " ".
+ // get_string('erroronline', 'error', $linenum) .". ".
+ // get_string('processingstops', 'error'),
+ // 'uploaduser.php?sesskey='.$USER->sesskey);
+ print_error('missingfield', 'error', 'uploaduser.php?sesskey='.$USER->sesskey, $name);

2 and 3) Needs a whole new string yes (in lang/en_utf8/forum.php)

4) yeah this is wierd, should be reverted