# HG changeset patch
# User František Kučera <franta-hg@frantovo.cz>
# Date 1276000920 -7200
# Node ID 4bd1e353c527cda797aef028ec4264a925d8ea4b
# Parent  2acdbc74bc2472d0aef9d1b4aa498456fecb2355
Kontrola komentářů i na výstupu.

diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-ejb/src/java/cz/frantovo/nekurak/dao/PodnikDAO.java
--- a/java/nekurak.net-ejb/src/java/cz/frantovo/nekurak/dao/PodnikDAO.java	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-ejb/src/java/cz/frantovo/nekurak/dao/PodnikDAO.java	Tue Jun 08 14:42:00 2010 +0200
@@ -1,5 +1,6 @@
 package cz.frantovo.nekurak.dao;
 
+import cz.frantovo.nekurak.dto.Komentar;
 import cz.frantovo.nekurak.dto.Podnik;
 import cz.frantovo.nekurak.dto.VysledekHlasovani;
 import cz.frantovo.superDAO.SuperDAO;
@@ -62,6 +63,14 @@
 	em.merge(p);
     }
 
+    public void zaloz(Komentar k) {
+	if (k.getDatum() == null) {
+	    k.setDatum(new Date());
+	}
+	
+	em.persist(k);
+    }
+
     /**
      * Zaznamená hlas uživatele, zda se v podniku má nebo nemá kouřit
      * @param podnik podnik o kterém se hlasuje
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-ejb/src/java/cz/frantovo/nekurak/ejb/PodnikEJB.java
--- a/java/nekurak.net-ejb/src/java/cz/frantovo/nekurak/ejb/PodnikEJB.java	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-ejb/src/java/cz/frantovo/nekurak/ejb/PodnikEJB.java	Tue Jun 08 14:42:00 2010 +0200
@@ -11,7 +11,6 @@
 import cz.frantovo.nekurak.vyjimky.KomentarovaVyjimka;
 import cz.frantovo.nekurak.vyjimky.TexyVyjimka;
 import java.util.Collection;
-import java.util.Date;
 import java.util.logging.Logger;
 import javax.annotation.Resource;
 import javax.annotation.security.RolesAllowed;
@@ -85,7 +84,7 @@
     @RolesAllowed("opravneny")
     public void komentuj(Komentar k) throws KomentarovaVyjimka, TexyVyjimka {
 	k.setUzivatel(ctx.getCallerPrincipal().getName());
-	k.setDatum(new Date());
+	k.setDatum(null);
 
 	/** Převedeme na XML */
 	switch (k.getTyp()) {
@@ -107,6 +106,6 @@
 
 
 	/** TODO: Uložíme do databáze */
-	log.severe("Komentář: " + k.getNadpis() + " | " + k.getKomentar() + " | " + k.getTyp());
+	podnikDAO.zaloz(k);
     }
 }
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-lib/src/cz/frantovo/nekurak/preklady_cs.properties
--- a/java/nekurak.net-lib/src/cz/frantovo/nekurak/preklady_cs.properties	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-lib/src/cz/frantovo/nekurak/preklady_cs.properties	Tue Jun 08 14:42:00 2010 +0200
@@ -93,5 +93,7 @@
 komentar.odeslat.chyba=Koment\u00E1\u0159 obsahuje nepovolen\u00E9 form\u00E1tov\u00E1n\u00ED.
 komentar.typ.prostyText=Prost\u00FD text
 komentar.typ.xhtml=XHTML form\u00E1tovan\u00FD text
+komentar.typ.xhtml.povoleneZnacky=Povolen\u00E9 zna\u010Dky jsou: &lt;p&gt;odstavec&lt;/p&gt;, &lt;strong&gt;tu\u010Dn\u011B&lt;/strong&gt;, &lt;em&gt;kurz\u00EDva&lt;/em&gt; a &lt;br/&gt;.
 komentar.typ.texy=Texy form\u00E1tovan\u00FD text
+komentar.nejdeZobrazit=Tento koment\u00E1\u0159 bohu\u017Eel nelze z technick\u00FDch d\u016Fvod\u016F zobrazit.
 
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-lib/src/cz/frantovo/nekurak/util/Komentare.java
--- a/java/nekurak.net-lib/src/cz/frantovo/nekurak/util/Komentare.java	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-lib/src/cz/frantovo/nekurak/util/Komentare.java	Tue Jun 08 14:42:00 2010 +0200
@@ -46,13 +46,13 @@
     }
 
     /**
-     * Přidá na konce řádků &lt;br/&gt;
+     * Převede konce řádků na &lt;br/&gt;
      * a celé zabalí do &lt;p/&gt;
-     * @param komentar
-     * @return
+     * @param komentar text se zalomenými řádky
+     * @return text s &lt;br/&gt; místo konců řádků
      */
     private static String zalomRadky(String komentar) {
-	return "<p>" + komentar.replaceAll("\n", "<br/>\n") + "</p>";
+	return "<p>" + komentar.replaceAll("\n", "<br/>") + "</p>";
     }
 
     /**
@@ -73,9 +73,9 @@
      * @param komentar
      * @return jestli komentář odpovídá
      */
-    public Document zkontroluj(String komentar) throws KomentarovaVyjimka {
+    public static Document zkontroluj(String komentar) throws KomentarovaVyjimka {
 	try {
-	    URL soubor = this.getClass().getClassLoader().getResource("cz/frantovo/nekurak/util/komentář.xsd");
+	    URL soubor = Komentare.class.getClassLoader().getResource("cz/frantovo/nekurak/util/komentář.xsd");
 	    SchemaFactory tovarnaSchemat = SchemaFactory.newInstance(XMLConstants.W3C_XML_SCHEMA_NS_URI);
 	    Schema schema = tovarnaSchemat.newSchema(soubor);
 
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-lib/test/cz/frantovo/nekurak/util/KomentareTest.java
--- a/java/nekurak.net-lib/test/cz/frantovo/nekurak/util/KomentareTest.java	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-lib/test/cz/frantovo/nekurak/util/KomentareTest.java	Tue Jun 08 14:42:00 2010 +0200
@@ -19,7 +19,7 @@
     public void upravProstýTextObal() {
 	String vystup = Komentare.upravProstyText("as\ndf");
 	vystup = Komentare.obal(vystup);
-	String pozadovanyVystup = "<div><p>as<br/>\ndf</p></div>";
+	String pozadovanyVystup = "<div><p>as<br/>df</p></div>";
 	assertEquals(pozadovanyVystup, vystup);
     }
 
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-web/src/java/cz/frantovo/nekurak/web/FunkceEL.java
--- a/java/nekurak.net-web/src/java/cz/frantovo/nekurak/web/FunkceEL.java	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-web/src/java/cz/frantovo/nekurak/web/FunkceEL.java	Tue Jun 08 14:42:00 2010 +0200
@@ -2,6 +2,10 @@
 
 import cz.frantovo.nekurak.dto.Podnik;
 import cz.frantovo.nekurak.servlet.Fotky;
+import cz.frantovo.nekurak.util.Komentare;
+import cz.frantovo.nekurak.vyjimky.KomentarovaVyjimka;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 /**
  * Knihovna funkcí použitelná v EL
@@ -20,4 +24,17 @@
     public static boolean maFotky(Podnik p) {
 	return !p.getFotky().isEmpty();
     }
+
+    /**
+     * @param komentar komentář včetně kořenové značky, XML jako text
+     * @return true, pokud vyhovuje XML schématu
+     */
+    public static boolean zkontrolujKomentar(String komentar) {
+	try {
+	    Komentare.zkontroluj(komentar);
+	    return true;
+	} catch (KomentarovaVyjimka ex) {
+	    return false;
+	}
+    }
 }
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-web/web/WEB-INF/casti/odeslatKomentar.jsp
--- a/java/nekurak.net-web/web/WEB-INF/casti/odeslatKomentar.jsp	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-web/web/WEB-INF/casti/odeslatKomentar.jsp	Tue Jun 08 14:42:00 2010 +0200
@@ -23,7 +23,7 @@
 
 	<c:choose>
 	    <c:when test="${odeslatKomentar.ok}">
-		<p>
+		<p class="informacniHlaska">
 		    <fmt:message key="komentar.odeslat.ok"/>,
 		    <a href="?akce=detail&amp;amp;podnik=${komentar.podnik}"><fmt:message key="komentar.odeslat.pokracujte"/></a>.
 		</p>
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-web/web/WEB-INF/nekurakFunkce.tld
--- a/java/nekurak.net-web/web/WEB-INF/nekurakFunkce.tld	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-web/web/WEB-INF/nekurakFunkce.tld	Tue Jun 08 14:42:00 2010 +0200
@@ -16,5 +16,11 @@
 	<function-class>cz.frantovo.nekurak.web.FunkceEL</function-class>
 	<function-signature>boolean maFotky(cz.frantovo.nekurak.dto.Podnik)</function-signature>
     </function>
+    <function>
+	<name>zkontrolujKomentar</name>
+	<description>Provede validaci komentáře oproti XML schématu</description>
+	<function-class>cz.frantovo.nekurak.web.FunkceEL</function-class>
+	<function-signature>boolean zkontrolujKomentar(java.lang.String)</function-signature>
+    </function>
 
 </taglib>
diff -r 2acdbc74bc24 -r 4bd1e353c527 java/nekurak.net-web/web/WEB-INF/tags/nekurak/komentareVypis.tag
--- a/java/nekurak.net-web/web/WEB-INF/tags/nekurak/komentareVypis.tag	Tue Jun 08 13:29:13 2010 +0200
+++ b/java/nekurak.net-web/web/WEB-INF/tags/nekurak/komentareVypis.tag	Tue Jun 08 14:42:00 2010 +0200
@@ -25,21 +25,22 @@
 		<p><a href="javascript:komentare.zobrazFormular()"><fmt:message key="komentare.komentovat"/></a></p>
 		<div id="formularKomentuj" style="display: none;">
 		    <form method="post" action="?akce=odeslatKomentar">
-		    <fieldset>
-			<input type="hidden" name="podnik" maxlength="255" value="${podnik.id}"/>
-			<label><fmt:message key="komentare.nadpis"/>: <input type="text" name="nadpis" maxlength="255"/></label>
-			<br/>
-			<textarea name="komentar" cols="30" rows="10"></textarea>
-			<br/>
-			<select name="typ">
-			    <option value="PROSTY_TEXT" selected="selected"><fmt:message key="komentar.typ.prostyText"/></option>
-			    <option value="XHTML"><fmt:message key="komentar.typ.xhtml"/></option>
-			    <option value="TEXY"><fmt:message key="komentar.typ.texy"/></option>
-			</select>
-			<br/>
-			<button value="submit"><fmt:message key="komentare.tlacitkoOdeslat"/></button>
-		    </fieldset>
-		</form>
+			<fieldset>
+			    <input type="hidden" name="podnik" maxlength="255" value="${podnik.id}"/>
+			    <label><fmt:message key="komentare.nadpis"/>: <input type="text" name="nadpis" maxlength="255"/></label>
+			    <br/>
+			    <textarea name="komentar" cols="30" rows="10"></textarea>
+			    <br/>
+			    <select name="typ">
+				<option value="PROSTY_TEXT" selected="selected"><fmt:message key="komentar.typ.prostyText"/></option>
+				<option value="XHTML"><fmt:message key="komentar.typ.xhtml"/></option>
+				<option value="TEXY"><fmt:message key="komentar.typ.texy"/></option>
+			    </select>
+			    <br/>
+			    <button value="submit"><fmt:message key="komentare.tlacitkoOdeslat"/></button>
+			</fieldset>
+		    </form>
+		    <p><fmt:message key="komentar.typ.xhtml.povoleneZnacky"/></p>
 		</div>
 	    </c:otherwise>
 	</c:choose>
@@ -50,11 +51,31 @@
 		<h4>
 		    <fmt:message key="komentare.napsal"/>: <c:out value="${k.uzivatel}" />,
 		    <fmt:formatDate value="${k.datum}" pattern="dd. MMMM yyyy HH:mm"/>
-		    <br/>
-		    <fmt:message key="komentare.nadpis"/>: <c:out value="${k.nadpis}" />
+		    <c:if test="${k.nadpis != null &amp;&amp; k.nadpis != ''}">
+			<br/>
+			<fmt:message key="komentare.nadpis"/>: <c:out value="${k.nadpis}" />
+		    </c:if>
 		</h4>
 		<!-- Pozor: předpokládáme, že v DB jsou pouze zkontrolovaná data -->
 		<c:out value="${k.komentar}" escapeXml="false" />
+
+		<!--
+		    Komentáře můžeme kontrolovat i tady
+		    pokud nemůžeme věřit, že v DB jsou zkontrolovaná data
+		-->
+		
+		<c:choose>
+		    <c:when test="${nkfn:zkontrolujKomentar(k.komentar)}">
+			<c:out value="${k.komentar}" escapeXml="false" />
+		    </c:when>
+		    <c:otherwise>
+			<!-- Špatně formátovaný komentář potom nezobrazíme -->
+			<p class="chybovaHlaska"><fmt:message key="komentar.nejdeZobrazit"/></p>
+		    </c:otherwise>
+		</c:choose>
+
+		<!-- Nebo ho jednoduše zobrazíme, ale escapujeme -->
+		<c:out value="${k.komentar}" escapeXml="${!nkfn:zkontrolujKomentar(k.komentar)}" />
 	    </div>
 	</c:forEach>
     </div>
diff -r 2acdbc74bc24 -r 4bd1e353c527 xml/komentář.xml
--- a/xml/komentář.xml	Tue Jun 08 13:29:13 2010 +0200
+++ b/xml/komentář.xml	Tue Jun 08 14:42:00 2010 +0200
@@ -18,4 +18,5 @@
 
 <p>…</p>
 
+
 </div>
diff -r 2acdbc74bc24 -r 4bd1e353c527 xml/komentář.xsd
--- a/xml/komentář.xsd	Tue Jun 08 13:29:13 2010 +0200
+++ b/xml/komentář.xsd	Tue Jun 08 14:42:00 2010 +0200
@@ -10,6 +10,7 @@
 			<xs:choice minOccurs="0" maxOccurs="unbounded">
 			    <xs:element name="em" type="xs:string"/>
 			    <xs:element name="strong" type="xs:string"/>
+			    <xs:element name="br"/>
 			</xs:choice>
 		    </xs:complexType>
 		</xs:element>
@@ -17,4 +18,4 @@
 	</xs:complexType>
     </xs:element>
 
-</xs:schema>
\ No newline at end of file
+</xs:schema>