From c15bc2e974019d14914ee01696cd2487f8908931 Mon Sep 17 00:00:00 2001
From: Art Cancro <ajc@uncensored.citadel.org>
Date: Mon, 23 May 2011 17:50:59 -0400
Subject: [PATCH] Revert "Generate UUID in blog posts to be used in permalinks"

This reverts commit fbeca38240b0a9771dc3c5349877e42768900025.
---
 citadel/modules/blog/serv_blog.c | 21 ++++++++++-----------
 libcitadel/lib/tools.c           |  5 +----
 webcit/blogview_renderer.c       | 12 +-----------
 3 files changed, 12 insertions(+), 26 deletions(-)

diff --git a/citadel/modules/blog/serv_blog.c b/citadel/modules/blog/serv_blog.c
index 5459898f7..6997a057e 100644
--- a/citadel/modules/blog/serv_blog.c
+++ b/citadel/modules/blog/serv_blog.c
@@ -1,11 +1,11 @@
 /*
  * Support for blog rooms
  *
- * Copyright (c) 1999-2011 by the citadel.org team
+ * Copyright (c) 1999-2010 by the citadel.org team
  *
- * This program is open source software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License as published
- * by the Free Software Foundation; either version 3 of the License, or
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful,
@@ -15,7 +15,7 @@
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
+ * Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA  02111-1307  USA
  */
 
 #include "sysdep.h"
@@ -59,10 +59,10 @@
 #include "ctdl_module.h"
 
 /*
- * Pre-save hook for saving a message in a blog room.
- * (Do we want to only do this for top-level messages?)
+ * sd sdhfksdjhkjsdfhk jsdhfkjsdfhkjsd hkfjhsdkjfhsdkjfhksdjfhsd
  */
 int blog_upload_beforesave(struct CtdlMessage *msg) {
+	char buf[SIZ];
 
 	/* Only run this hook for blog rooms */
 	if (CC->room.QRdefaultview != VIEW_BLOG) {
@@ -72,11 +72,10 @@ int blog_upload_beforesave(struct CtdlMessage *msg) {
 	/* 
 	 * If the message doesn't have an EUID, give it one.
 	 */
-	if (msg->cm_fields['E'] == NULL)
+	if (msg->cm_fields['E'] != NULL)
 	{
-		char uuid[37];
-		generate_uuid(uuid);
-		msg->cm_fields['E'] = strdup(uuid);
+		generate_uuid(buf);
+		msg->cm_fields['E'] = strdup(buf);
 	}
 
 	/* Now allow the save to complete. */
diff --git a/libcitadel/lib/tools.c b/libcitadel/lib/tools.c
index 0c012e795..db8fe3af8 100644
--- a/libcitadel/lib/tools.c
+++ b/libcitadel/lib/tools.c
@@ -858,11 +858,8 @@ char *strcpy(char *dest, const char *src) {
  * Generate a new, globally unique UID parameter for a calendar etc. object
  */
 void generate_uuid(char *buf) {
-	static int seq = (-1);
+	static int seq = 0;
 
-	if (seq == (-1)) {
-		seq = (int)rand();
-	}
 	++seq;
 	seq = (seq % 0x0FFF) ;
 
diff --git a/webcit/blogview_renderer.c b/webcit/blogview_renderer.c
index 8190cb8c8..215fa94ef 100644
--- a/webcit/blogview_renderer.c
+++ b/webcit/blogview_renderer.c
@@ -87,7 +87,6 @@ void blogpost_render_and_destroy(struct blogpost *bp) {
 struct bltr {
 	int id;
 	int refs;
-	StrBuf *euid;
 };
 
 
@@ -124,7 +123,7 @@ struct bltr blogview_learn_thread_references(long msgnum)
 	StrBuf *Buf;
 	StrBuf *r;
 	int len;
-	struct bltr bltr = { 0, 0, NULL };
+	struct bltr bltr = { 0, 0 } ;
 	Buf = NewStrBuf();
 	r = NewStrBuf();
 	serv_printf("MSG0 %ld|1", msgnum);		/* top level citadel headers only */
@@ -143,10 +142,6 @@ struct bltr blogview_learn_thread_references(long msgnum)
 				StrBufExtract_token(r, Buf, 0, '|');
 				bltr.refs = HashLittle(ChrPtr(r), StrLength(r));
 			}
-			else if (!strncasecmp(ChrPtr(Buf), "exti=", 5)) {
-				StrBufCutLeft(Buf, 5);		/* trim the field name */
-				bltr.euid = NewStrBufDup(Buf);
-			}
 		}
 	}
 	FreeStrBuf(&Buf);
@@ -170,11 +165,6 @@ int blogview_LoadMsgFromServer(SharedMessageStatus *Stat,
 
 	b = blogview_learn_thread_references(Msg->msgnum);
 
-	if (b.euid != NULL) {
-		syslog(LOG_DEBUG, "\033[7m%s\033[0m", ChrPtr(b.euid));
-		FreeStrBuf(&b.euid);
-	}
-
 	/* FIXME an optimization here -- one we ought to perform -- is to exit this
 	 * function immediately if the viewer is only interested in a single post and
 	 * that message ID is neither the id nor the refs.  Actually, that might *be*
-- 
2.30.2