Merge branch 'maint-rel-1.6' of https://github.com/vivo-project/Vitro into maint-rel-1.6

webapp/config/example.developer.properties

@@ -32,6 +32,7 @@
 #
 # developer.permitAnonymousControl
 
+
 #------------------------------------------------------------------------------
 # Freemarker 
 #------------------------------------------------------------------------------
@@ -52,6 +53,25 @@
 #
 # developer.defeatFreemarkerCache = true
 
+
+#------------------------------------------------------------------------------
+# Page configuration 
+#------------------------------------------------------------------------------
+
+#
+# Turn on logging of custom list view configuration files. Each time a property
+# uses a list view other than the default, note it in the log. The default is 
+# 'false'.
+#
+# developer.pageContents.logCustomListView = true
+
+#
+# Turn on logging of custom short views. Each time an individual uses a short
+# view other than the default, note it in the log. The default is 'false'.
+#
+# developer.pageContents.logCustomShortView = true
+
+
 #------------------------------------------------------------------------------
 # Internationalization 
 #------------------------------------------------------------------------------
@@ -98,10 +118,18 @@
 # developer.loggingRDFService.stackTrace = true
 
 #
-# If SPARQL query logging is enabled, a regular expression can be used to 
-# restrict the number of entries that are produced. The expression is
-# tested against each line in the (unabridged) stack trace. If the 
-# expression doesn't match any line in the stack trace, then no log entry
-# is made. The default is 'false'.
+# If SPARQL query logging is enabled, restrict the number of log entries by
+# matching a regular expression against the query string. If the expression 
+# doesn't match the string, then no log entry is made. The default is "", 
+# which means no restriction.
 #
-# developer.loggingRDFService.restriction = true
+# developer.loggingRDFService.queryRestriction = .*
+
+#
+# If SPARQL query logging is enabled, restrict the number of log entries by
+# matching a regular expression against the stack trace. The abridged stack 
+# trace is concatenated into a single string of fully qualified class names 
+# and method names. If the expression doesn't match the string, then no log 
+# entry is made. The default is "", which means no restriction.
+#
+# developer.loggingRDFService.stackRestriction = .*

webapp/languages/es_GO/i18n/all_es_GO.properties

@@ -481,6 +481,9 @@ restrict_logins_mixed_caps = Restringir conexiones
 site_information = Información del sitio
 user_accounts = Las cuentas de usuario
 
+activate_developer_panel = Activar el panel desarrollador
+activate_developer_panel_mixed_caps = Activar el panel desarrollador
+
 #
 # search controller ( PagedSearchController.java )
 #

webapp/src/edu/cornell/mannlib/vitro/webapp/controller/freemarker/BaseSiteAdminController.java

@@ -73,6 +73,10 @@ public class BaseSiteAdminController extends FreemarkerHttpServlet {
 			urls.put("rebuildSearchIndex", UrlBuilder.getUrl("/SearchIndex"));
 		}
 		
+		if (PolicyHelper.isAuthorizedForActions(vreq, SimplePermission.ENABLE_DEVELOPER_PANEL.ACTIONS)) {
+			urls.put("activateDeveloperPanel", "javascript:new DeveloperPanel(developerAjaxUrl).setupDeveloperPanel({developerEnabled: true});");
+		}
+		
         return urls;
     }
 

webapp/src/edu/cornell/mannlib/vitro/webapp/freemarker/loader/FreemarkerTemplateLoader.java

@@ -15,6 +15,8 @@ import java.nio.file.attribute.BasicFileAttributes;
 import java.util.Comparator;
 import java.util.SortedSet;
 import java.util.TreeSet;
+import java.util.regex.Matcher;
+import java.util.regex.Pattern;
 
 import org.apache.commons.lang.StringUtils;
 import org.apache.commons.logging.Log;
@@ -178,64 +180,71 @@ public class FreemarkerTemplateLoader implements TemplateLoader {
 	 * search term, and how well they match.
 	 */
 	static class PathPieces {
+		static final Pattern PATTERN = Pattern.compile("(.+?)" // base name
+				+ "(_[a-z]{2})?" // optional language
+				+ "(_[A-Z]{2})?" // optional country
+				+ "(\\.\\w+)?" // optional extension
+		);
+
 		final Path path;
 		final String base;
 		final String language;
 		final String region;
 		final String extension;
 
-		public PathPieces(String searchTerm) {
-			this(Paths.get(searchTerm));
+		public PathPieces(String pathString) {
+			this(Paths.get(pathString));
 		}
 
 		public PathPieces(Path path) {
 			this.path = path;
 
 			String filename = path.getFileName().toString();
-			int dotHere = filename.lastIndexOf('.');
-			String basename;
-			if (dotHere != -1) {
-				basename = filename.substring(0, dotHere);
-				this.extension = filename.substring(dotHere);
-			} else {
-				basename = filename;
-				this.extension = "";
-			}
 
-			int break2 = basename.lastIndexOf('_');
-			int break1 = basename.lastIndexOf('_', break2 - 1);
-			if (break1 != -1) {
-				this.base = basename.substring(0, break1);
-				this.language = basename.substring(break1, break2);
-				this.region = basename.substring(break2);
-			} else if (break2 != -1) {
-				this.base = basename.substring(0, break2);
-				this.language = basename.substring(break2);
-				this.region = "";
+			Matcher m = PATTERN.matcher(filename);
+			if (m.matches()) {
+				base = getGroup(m, 1);
+				language = getGroup(m, 2);
+				region = getGroup(m, 3);
+				extension = getGroup(m, 4);
 			} else {
-				this.base = basename;
-				this.language = "";
-				this.region = "";
+				base = filename;
+				language = "";
+				region = "";
+				extension = "";
 			}
 		}
 
-		/** This is the search term. Does that candidate qualify as a result? */
+		private String getGroup(Matcher m, int i) {
+			return (m.start(i) == -1) ? "" : m.group(i);
+		}
+
+		/**
+		 * If I'm searching for this, is that an acceptable match?
+		 * 
+		 * Note that this is asymetrical -- a search term without a region will
+		 * match a candidate with a region, but not vice versa. Same with
+		 * language.
+		 */
 		public boolean matches(PathPieces that) {
 			return base.equals(that.base) && extension.equals(that.extension)
 					&& (language.isEmpty() || language.equals(that.language))
 					&& (region.isEmpty() || region.equals(that.region));
 		}
 
+		/**
+		 * How good a match is that to this?
+		 */
 		public int score(PathPieces that) {
 			if (matches(that)) {
 				if (that.language.equals(language)) {
 					if (that.region.equals(region)) {
-						return 3; // match language and region
+						return 3; // exact match.
 					} else {
-						return 2; // match language, default region.
+						return 2; // same language, approximate region.
 					}
 				} else {
-					return 1; // default language.
+					return 1; // approximate language.
 				}
 			} else {
 				return -1; // doesn't match.

webapp/src/edu/cornell/mannlib/vitro/webapp/rdfservice/impl/logging/RDFServiceLogger.java

@@ -23,9 +23,12 @@ import edu.cornell.mannlib.vitro.webapp.utils.developer.DeveloperSettings.Keys;
  * 
  * If not enabled, or if the logging level is insufficient, this does nothing.
  * 
- * If enabled, it checks for restrictions. If there is a restriction pattern
- * (regular expression), the a log message will only be printed if one of the
- * fully-qualified class names in the stack trace matches that pattern.
+ * If enabled, it checks for restrictions. If there is a restriction on the call
+ * stack (regular expression), then a log message will only be printed if the
+ * pattern is found in the concatenated call stack (fully-qualified class names
+ * and method names). If there is a restriction on the query string (regular
+ * expression) then a log message will only be printed if the pattern is found
+ * in the query string.
  * 
  * If everything passes muster, the constructor will record the time that the
  * instance was created.
@@ -47,7 +50,8 @@ public class RDFServiceLogger implements AutoCloseable {
 
 	private boolean isEnabled;
 	private boolean traceRequested;
-	private Pattern restriction;
+	private Pattern queryStringRestriction;
+	private Pattern callStackRestriction;
 
 	private String methodName;
 	private List<StackTraceElement> trace = Collections.emptyList();
@@ -62,7 +66,7 @@ public class RDFServiceLogger implements AutoCloseable {
 
 		if (isEnabled && log.isInfoEnabled()) {
 			loadStackTrace();
-			if (passesRestrictions()) {
+			if (passesQueryRestriction() && passesStackRestriction()) {
 				this.startTime = System.currentTimeMillis();
 			}
 		}
@@ -72,20 +76,23 @@ public class RDFServiceLogger implements AutoCloseable {
 		DeveloperSettings settings = DeveloperSettings.getBean(ctx);
 		isEnabled = settings.getBoolean(Keys.LOGGING_RDF_ENABLE);
 		traceRequested = settings.getBoolean(Keys.LOGGING_RDF_STACK_TRACE);
-
-		String restrictionString = settings
-				.getString(Keys.LOGGING_RDF_RESTRICTION);
-		if (StringUtils.isBlank(restrictionString)) {
-			restriction = null;
-		} else {
-			try {
-				restriction = Pattern.compile(restrictionString);
-			} catch (Exception e) {
-				log.error("Failed to compile the pattern for "
-						+ Keys.LOGGING_RDF_RESTRICTION + " = " + restriction
-						+ " " + e);
-				isEnabled = false;
+		queryStringRestriction = patternFromSettings(settings,
+				Keys.LOGGING_RDF_QUERY_RESTRICTION);
+		callStackRestriction = patternFromSettings(settings,
+				Keys.LOGGING_RDF_STACK_RESTRICTION);
 	}
+
+	private Pattern patternFromSettings(DeveloperSettings settings, Keys key) {
+		String patternString = settings.getString(key);
+		if (StringUtils.isBlank(patternString)) {
+			return null;
+		}
+		try {
+			return Pattern.compile(patternString);
+		} catch (Exception e) {
+			log.error("Failed to compile the pattern for " + key + " = "
+					+ patternString + " " + e);
+			return Pattern.compile("^_____NEVER MATCH_____$");
 		}
 	}
 
@@ -144,16 +151,39 @@ public class RDFServiceLogger implements AutoCloseable {
 		}
 	}
 
-	private boolean passesRestrictions() {
-		if (restriction == null) {
+	private boolean passesQueryRestriction() {
+		if (queryStringRestriction == null) {
 			return true;
 		}
+		String q = assembleQueryString();
+		return queryStringRestriction.matcher(q).find();
+	}
+
+	private String assembleQueryString() {
+		StringBuilder query = new StringBuilder();
+		for (Object arg : args) {
+			if (arg instanceof String) {
+				query.append((String) arg).append(" ");
+			}
+		}
+		return query.deleteCharAt(query.length() - 1).toString();
+	}
+
+	private boolean passesStackRestriction() {
+		if (callStackRestriction == null) {
+			return true;
+		}
+		String q = assembleCallStackString();
+		return callStackRestriction.matcher(q).find();
+	}
+
+	private String assembleCallStackString() {
+		StringBuilder stack = new StringBuilder();
 		for (StackTraceElement ste : trace) {
-			if (restriction.matcher(ste.getClassName()).find()) {
-				return true;
+			stack.append(ste.getClassName()).append(" ")
+					.append(ste.getMethodName()).append(" ");
 		}
-		}
-		return false;
+		return stack.deleteCharAt(stack.length() - 1).toString();
 	}
 
 	@Override

webapp/src/edu/cornell/mannlib/vitro/webapp/services/shortview/FakeApplicationOntologyService.java

@@ -380,6 +380,12 @@ public class FakeApplicationOntologyService {
 			return dataGetters;
 		}
 
+		@Override
+		public String toString() {
+			return "[template=" + templateName + ", dataGetters=" + dataGetters
+					+ "]";
+		}
+
 	}
 
 	/** The view specifications that we read from the config file. */

webapp/src/edu/cornell/mannlib/vitro/webapp/services/shortview/ShortViewLogger.java

@@ -0,0 +1,47 @@
+/* $This file is distributed under the terms of the license in /doc/license.txt$ */
+
+package edu.cornell.mannlib.vitro.webapp.services.shortview;
+
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+import edu.cornell.mannlib.vitro.webapp.beans.Individual;
+import edu.cornell.mannlib.vitro.webapp.controller.VitroRequest;
+import edu.cornell.mannlib.vitro.webapp.services.shortview.FakeApplicationOntologyService.TemplateAndDataGetters;
+import edu.cornell.mannlib.vitro.webapp.utils.developer.DeveloperSettings;
+import edu.cornell.mannlib.vitro.webapp.utils.developer.DeveloperSettings.Keys;
+
+/**
+ * When we use a short view other than the default, log it.
+ */
+public class ShortViewLogger {
+	private static final Log log = LogFactory.getLog(ShortViewLogger.class);
+
+	public static void log(VitroRequest vreq, String contextName,
+			Individual individual, String classUri, TemplateAndDataGetters tdg) {
+		if (isLogging(vreq)) {
+			log.info("Using custom short view in " + contextName + " because '"
+					+ individual.getURI() + "' (" + individual.getLabel()
+					+ ") has type '" + classUri + "': " + tdg);
+		}
+	}
+
+	public static void log(VitroRequest vreq, String contextName,
+			Individual individual) {
+		if (isLogging(vreq)) {
+			log.info("Using default short view in " + contextName + " for '"
+					+ individual.getURI() + "' (" + individual.getLabel() + ")");
+		}
+	}
+
+	private static boolean isLogging(VitroRequest vreq) {
+		if (!log.isInfoEnabled()) {
+			return false;
+		}
+		DeveloperSettings settings = DeveloperSettings.getBean(vreq);
+		return settings.getBoolean(Keys.ENABLED)
+				&& settings
+						.getBoolean(Keys.PAGE_CONTENTS_LOG_CUSTOM_SHORT_VIEW);
+	}
+
+}

webapp/src/edu/cornell/mannlib/vitro/webapp/services/shortview/ShortViewServiceImpl.java

@@ -15,9 +15,7 @@ import org.apache.commons.logging.Log;
 import org.apache.commons.logging.LogFactory;
 
 import edu.cornell.mannlib.vitro.webapp.beans.Individual;
-import edu.cornell.mannlib.vitro.webapp.beans.ObjectPropertyStatement;
 import edu.cornell.mannlib.vitro.webapp.controller.VitroRequest;
-import edu.cornell.mannlib.vitro.webapp.dao.VitroVocabulary;
 import edu.cornell.mannlib.vitro.webapp.services.freemarker.FreemarkerProcessingService;
 import edu.cornell.mannlib.vitro.webapp.services.freemarker.FreemarkerProcessingService.TemplateParsingException;
 import edu.cornell.mannlib.vitro.webapp.services.freemarker.FreemarkerProcessingService.TemplateProcessingException;
@@ -116,11 +114,13 @@ public class ShortViewServiceImpl implements ShortViewService {
 			TemplateAndDataGetters tdg = faker.getShortViewProperties(vreq,
 					individual, classUri, svContext.name());
 			if (tdg != null) {
+				ShortViewLogger.log(vreq, svContext.name(), individual, classUri, tdg);
 				return tdg;
 			}
 		}
 
 		// Didn't find one? Use the default values.
+		ShortViewLogger.log(vreq, svContext.name(), individual);
 		return new TemplateAndDataGetters(svContext.getDefaultTemplateName());
 	}
 

webapp/src/edu/cornell/mannlib/vitro/webapp/utils/developer/DeveloperSettings.java

@@ -82,8 +82,28 @@ public class DeveloperSettings {
 		 * Don't log with the LoggingRDFService unless the calling stack meets
 		 * this restriction.
 		 */
-		LOGGING_RDF_RESTRICTION("developer.loggingRDFService.restriction",
-				false);
+		LOGGING_RDF_QUERY_RESTRICTION(
+				"developer.loggingRDFService.queryRestriction", false),
+
+		/**
+		 * Don't log with the LoggingRDFService unless the calling stack meets
+		 * this restriction.
+		 */
+		LOGGING_RDF_STACK_RESTRICTION(
+				"developer.loggingRDFService.stackRestriction", false),
+
+		/**
+		 * Tell the CustomListViewLogger to note the use of non-default custom
+		 * list views.
+		 */
+		PAGE_CONTENTS_LOG_CUSTOM_LIST_VIEW(
+				"developer.pageContents.logCustomListView", true),
+
+		/**
+		 * Tell the ShortViewLogger to note the use of non-default short views.
+		 */
+		PAGE_CONTENTS_LOG_CUSTOM_SHORT_VIEW(
+				"developer.pageContents.logCustomShortView", true);
 		
 		private final String propertyName;
 		private final String elementId;

webapp/src/edu/cornell/mannlib/vitro/webapp/web/templatemodels/customlistview/CustomListViewLogger.java

@@ -0,0 +1,38 @@
+/* $This file is distributed under the terms of the license in /doc/license.txt$ */
+
+package edu.cornell.mannlib.vitro.webapp.web.templatemodels.customlistview;
+
+import org.apache.commons.logging.Log;
+import org.apache.commons.logging.LogFactory;
+
+import edu.cornell.mannlib.vitro.webapp.beans.ObjectProperty;
+import edu.cornell.mannlib.vitro.webapp.controller.VitroRequest;
+import edu.cornell.mannlib.vitro.webapp.utils.developer.DeveloperSettings;
+import edu.cornell.mannlib.vitro.webapp.utils.developer.DeveloperSettings.Keys;
+
+/**
+ * If enabled in the developer settings (and log levels), log every non-default
+ * custom list view.
+ */
+public class CustomListViewLogger {
+	private static final Log log = LogFactory
+			.getLog(CustomListViewLogger.class);
+
+	public static void log(VitroRequest vreq, ObjectProperty op,
+			String configFileName) {
+		if (isLogging(vreq)) {
+			log.info("Using list view: '" + configFileName + "' for "
+					+ op.getURI() + " (" + op.getLabel() + ")");
+
+		}
+	}
+
+	private static boolean isLogging(VitroRequest vreq) {
+		if (!log.isInfoEnabled()) {
+			return false;
+		}
+		DeveloperSettings settings = DeveloperSettings.getBean(vreq);
+		return settings.getBoolean(Keys.ENABLED)
+				&& settings.getBoolean(Keys.PAGE_CONTENTS_LOG_CUSTOM_LIST_VIEW);
+	}
+}

webapp/src/edu/cornell/mannlib/vitro/webapp/web/templatemodels/customlistview/PropertyListConfig.java

@@ -61,6 +61,8 @@ public class PropertyListConfig {
         String configFileName = wadf.getObjectPropertyDao().getCustomListViewConfigFileName(op);
         if (configFileName == null) { // no custom config; use default config
             configFileName = DEFAULT_CONFIG_FILE_NAME;
+        } else {
+        	CustomListViewLogger.log(vreq, op, configFileName);
         }
         log.debug("Using list view config file " + configFileName + " for object property " + op.getURI());
         

webapp/test/edu/cornell/mannlib/vitro/webapp/freemarker/loader/FreemarkerTemplateLoaderTest.java

@@ -286,18 +286,20 @@ public class FreemarkerTemplateLoaderTest {
 	 * would.
 	 */
 	private void assertFM(String searchTerm, int expectedNumberOfTries,
-			String expectedBestFit) {
+			String expectedBestString) {
+		Path expectedBestFit = expectedBestString == null ? null : Paths
+				.get(expectedBestString);
 		PathPieces stPp = new PathPieces(searchTerm);
 
 		int actualNumberOfTries = 0;
-		String actualBestFit = null;
+		Path actualBestFit = null;
 
 		if (StringUtils.isNotBlank(stPp.region)) {
 			actualNumberOfTries++;
 			SortedSet<PathPieces> matches = runTheVisitor(stPp.base
 					+ stPp.language + stPp.region + stPp.extension);
 			if (!matches.isEmpty()) {
-				actualBestFit = matches.last().path.toString();
+				actualBestFit = matches.last().path;
 			}
 		}
 		if (actualBestFit == null && StringUtils.isNotBlank(stPp.language)) {
@@ -305,7 +307,7 @@ public class FreemarkerTemplateLoaderTest {
 			SortedSet<PathPieces> matches = runTheVisitor(stPp.base
 					+ stPp.language + stPp.extension);
 			if (!matches.isEmpty()) {
-				actualBestFit = matches.last().path.toString();
+				actualBestFit = matches.last().path;
 			}
 		}
 		if (actualBestFit == null) {
@@ -313,7 +315,7 @@ public class FreemarkerTemplateLoaderTest {
 			SortedSet<PathPieces> matches = runTheVisitor(stPp.base
 					+ stPp.extension);
 			if (!matches.isEmpty()) {
-				actualBestFit = matches.last().path.toString();
+				actualBestFit = matches.last().path;
 			}
 		}
 

webapp/web/i18n/all.properties

@@ -493,6 +493,9 @@ restrict_logins_mixed_caps = Restrict logins
 site_information = Site information
 user_accounts = User accounts
 
+activate_developer_panel = Activate developer panel
+activate_developer_panel_mixed_caps = Activate developer panel
+
 #
 # search controller ( PagedSearchController.java )
 #

webapp/web/js/developer/developerPanel.js

@@ -1,6 +1,6 @@
 /* $This file is distributed under the terms of the license in /doc/license.txt$ */
 
- function DeveloperPanel(developerAjaxUrl) {
+function DeveloperPanel(developerAjaxUrl) {
 	this.setupDeveloperPanel = updateDeveloperPanel;
 	
 	function updateDeveloperPanel(data) {
@@ -42,13 +42,16 @@
 		var developerEnabled = document.getElementById("developerEnabled").checked;
 		document.getElementById("developerDefeatFreemarkerCache").disabled = !developerEnabled;
 		document.getElementById("developerInsertFreemarkerDelimiters").disabled = !developerEnabled;
+		document.getElementById("developerPageContentsLogCustomListView").disabled = !developerEnabled;
+		document.getElementById("developerPageContentsLogCustomShortView").disabled = !developerEnabled;
 		document.getElementById("developerI18nDefeatCache").disabled = !developerEnabled;
 		document.getElementById("developerI18nLogStringRequests").disabled = !developerEnabled;
 		document.getElementById("developerLoggingRDFServiceEnable").disabled = !developerEnabled;
 	
 		var rdfServiceEnabled = developerEnabled && document.getElementById("developerLoggingRDFServiceEnable").checked;
 		document.getElementById("developerLoggingRDFServiceStackTrace").disabled = !rdfServiceEnabled;
-		document.getElementById("developerLoggingRDFServiceRestriction").disabled = !rdfServiceEnabled;
+		document.getElementById("developerLoggingRDFServiceQueryRestriction").disabled = !rdfServiceEnabled;
+		document.getElementById("developerLoggingRDFServiceStackRestriction").disabled = !rdfServiceEnabled;
 	}
 
 	function collectFormData() {
@@ -56,11 +59,14 @@
 		getCheckbox("developerEnabled", data);
 		getCheckbox("developerDefeatFreemarkerCache", data);
 		getCheckbox("developerInsertFreemarkerDelimiters", data);
+		getCheckbox("developerPageContentsLogCustomListView", data);
+		getCheckbox("developerPageContentsLogCustomShortView", data);
 		getCheckbox("developerI18nDefeatCache", data);
 		getCheckbox("developerI18nLogStringRequests", data);
 		getCheckbox("developerLoggingRDFServiceEnable", data);
 		getCheckbox("developerLoggingRDFServiceStackTrace", data);
-		getText("developerLoggingRDFServiceRestriction", data);
+		getText("developerLoggingRDFServiceQueryRestriction", data);
+		getText("developerLoggingRDFServiceStackRestriction", data);
 		return data;
 	}
 
@@ -79,3 +85,4 @@
 $(document).ready(function() {   
 	new DeveloperPanel(developerAjaxUrl).setupDeveloperPanel({});	
 }); 
+

webapp/web/templates/freemarker/body/siteAdmin/siteAdmin-indexCacheRebuild.ftl

@@ -18,6 +18,10 @@
             <#if indexCacheRebuild.recomputeInferences?has_content>
                 <li role="listitem"><a href="${indexCacheRebuild.recomputeInferences}" title="${i18n().recompute_inferences}">${i18n().recompute_inferences_mixed_caps}</a></li>
             </#if>
+            
+            <#if indexCacheRebuild.activateDeveloperPanel?has_content>
+                <li role="listitem"><a href="${indexCacheRebuild.activateDeveloperPanel}" title="${i18n().activate_developer_panel}">${i18n().activate_developer_panel_mixed_caps}</a></li>
+            </#if>
         </ul>
     </section>
 </#if>

webapp/web/templates/freemarker/page/partials/developerPanel.ftl

@@ -6,7 +6,7 @@
 </#macro>
 
 <#macro showTextbox key>
-	<input type="text" id="${key}" size="40" value="${settings[key]}" >
+	<input type="text" id="${key}" size="30" value="${settings[key]}" >
 </#macro>
 
 
@@ -14,17 +14,40 @@
 div.developer {
 	background-color: #f7dd8a; 
 	padding: 0px 10px 0px 10px;
-	font-size: small;
 	font-variant: small-caps;
 }
 
 div.developer #developerPanelBody {
 	display: none;
+	line-height: 1em;
+	font-size: small;
 }
 
-div.developer .container {
-	border: thin groove black
+div.developer div.devleft {
+	width: 49%
 }
+
+div.developer div.devright {
+	float: right;
+	width: 49%
+}
+
+div.developer div.container {
+	border: thin groove black;
+	padding: 3px 10px 0px 10px;
+	margin: 3px 0px 3px 0px;
+}
+
+div.developer div.within {
+	padding-left: 1em;
+}
+
+div.developer input[type="text"] { 
+	padding: 2px 10px 2px 10px; 
+	line-height: 1em;
+	margin: 2px 2px 2px 2px; 
+	}
+
 </style>
 
 <#if !settings.developerEnabled>
@@ -38,12 +61,40 @@ div.developer .container {
 			<span id="developerPanelClickText">(click for Options)</span>
 		</h1>
 		<div id="developerPanelBody">
-			<form>
+			<div>
 				<label>
 					<@showCheckbox "developerEnabled" />
 					Enable developer mode
 				</label>
+			</div>
 			
+			<div class="devright">
+				<div class="container">
+					Page configuration
+					<label>
+						<@showCheckbox "developerPageContentsLogCustomListView" />
+						Log the use of custom list view XML files.
+					</label>
+					<label>
+						<@showCheckbox "developerPageContentsLogCustomShortView" />
+						Log the use of custom short views in search, index and browse pages.
+					</label>
+				</div>
+				
+				<div class="container">
+					Language support
+					<label>
+						<@showCheckbox "developerI18nDefeatCache" />
+						Defeat the cache of language property files
+					</label>
+					<label>
+						<@showCheckbox "developerI18nLogStringRequests" />
+						Log the retrieval of language strings
+					</label>
+				</div>
+			</div>
+				
+			<div class="devleft">
 				<div class="container">
 					Freemarker templates
 					<label>
@@ -62,30 +113,26 @@ div.developer .container {
 						<@showCheckbox "developerLoggingRDFServiceEnable" />
 						Log each query 
 					</label>
+					<div class="within">
 						<label>
 							<@showCheckbox "developerLoggingRDFServiceStackTrace" />
 							Add stack trace
 						</label>
+						<label>
+							Restrict by query string
+							<@showTextbox "developerLoggingRDFServiceQueryRestriction" />
+						</label>
 						<label>
 							Restrict by calling stack
-						<@showTextbox "developerLoggingRDFServiceRestriction" />
+							<@showTextbox "developerLoggingRDFServiceStackRestriction" />
 						</label>
 					</div>
-
-				<div class="container">
-					Language support
-					<label>
-						<@showCheckbox "developerI18nDefeatCache" />
-						Defeat the cache of language property files
-					</label>
-					<label>
-						<@showCheckbox "developerI18nLogStringRequests" />
-						Log the retrieval of language strings
-					</label>
+				</div>
 			</div>
 
+			<div>
 				<input type="button" id="developerPanelSaveButton" value="Save Settings" name="foo" />
-			</form>
+			</div>
 		</div>
 	</div>
 </#if>