Index: indexforms-user.inc.php
===================================================================
--- indexforms-user.inc.php	(revision 617)
+++ indexforms-user.inc.php	(working copy)
@@ -1012,12 +1012,12 @@
 	// TODO - prob better to have these actions in user_admin instead.
 	
 	/* add a macro? */
-	if (!empty($_GET['macroNewName']))
-		user_macro_create($_GET['macroUsername'], $_GET['macroNewName'],$_GET['macroNewBody']); 
+	if (!empty($_GET['macroNewName']) && !empty($_GET['macroNewBody']) )
+		user_macro_create($User->username, $_GET['macroNewName'],$_GET['macroNewBody']); 
 	
 	/* delete a macro? */
-	if (!empty($_GET['macroDelete']))
-		user_macro_delete($_GET['macroUsername'], $_GET['macroDelete'],$_GET['macroDeleteNArgs']);
+	if (!empty($_GET['macroDelete']) && !empty($_GET['macroDeleteNArgs']))
+		user_macro_delete($User->username, $_GET['macroDelete'], $_GET['macroDeleteNArgs']);
 	// TODO use ID field instead
 	
 	?>
@@ -1067,7 +1067,6 @@
 				, '<input type="submit" value="Delete macro" /></td>'
 				, '<input type="hidden" name="macroDelete" value="'.$r->macro_name.'" />'
 				, '<input type="hidden" name="macroDeleteNArgs" value="'.$r->macro_num_args.'" />'
-				, '<input type="hidden" name="macroUsername" value="'.$User->username.'" />'
 				, '<input type="hidden" name="thisQ" value="userSettings" />'
 				, '<input type="hidden" name="uT" value="y" />'
 				, '</form>';
@@ -1103,7 +1102,7 @@
 		</tr>
 		
 		<input type="hidden" name="macroUsername" value="<?php echo $Uxer->username;?>" />
-		<input type="hidden" name="thisQ" value="userSettings" />
+		<input type="hidden" name="thisQ" value="userMacros" />
 		<input type="hidden" name="uT" value="y" />
 		
 	</form>
Index: indexforms-saved.inc.php
===================================================================
--- indexforms-saved.inc.php	(revision 620)
+++ indexforms-saved.inc.php	(working copy)
@@ -788,7 +788,7 @@
 
 		<tr>
 			<td class="concordgrey">Date created:</td>
-			<td class="concordgeneral"><?php echo date($matrix->create_time); ?></td>
+			<td class="concordgeneral"><?php echo date(DATE_RSS, $matrix->create_time); ?></td>
 		</tr>	
 
 		<tr>
@@ -829,6 +829,9 @@
 			foreach($variable_list as $v)
 				echo "\n\t\t<tr>"
 					, '<td class="concordgeneral">' , $v->label , '</td>'
+					
+					// TODO if source_info is too long, shorten it and add a Jquery "click for more" that will un-hide it.
+					
 					, '<td class="concordgeneral">' , cqpweb_htmlspecialchars($v->source_info) , '</td>'
 					, "</tr>\n" 
 					;