Just 2 small edits to comments

root/includes/acp/acp_portal.php

@@ -155,7 +155,7 @@ class acp_portal
 				{
 					$submit = false;
 				}
-
+				
 				// We go through the display_vars to make sure no one is trying to set variables he/she is not allowed to...
 				foreach ($display_vars['vars'] as $config_name => $null)
 				{
@@ -222,6 +222,7 @@ class acp_portal
 						'module_group_ids'		=> $module_permission,
 					);
 					
+					// check if module image file actually exists
 					check_file_src($sql_ary['module_image_src'], '', $module_id);
 
 					$sql = 'UPDATE ' . PORTAL_MODULES_TABLE . '

root/language/en/mods/portal.php

@@ -32,7 +32,6 @@ if (empty($lang) || !is_array($lang))
 // You do not need this where single placeholders are used, e.g. 'Message %d' is fine
 // equally where a string contains only two placeholders which are used to wrap text
 // in a url you again do not need to specify an order e.g., 'Click %sHERE%s' is fine
-// @todo: merge into mods/portal/
 
 // Common
 $lang = array_merge($lang, array(