From 871395b74470a70e37aeb684b820e289b0930cba Mon Sep 17 00:00:00 2001
From: Cash Costello <cash.costello@gmail.com>
Date: Sat, 28 Mar 2009 14:10:18 +0000
Subject: fixed bug with access levels and improved image and album editing

---
 views/default/tidypics/forms/edit.php | 82 +++++++++++++++--------------------
 1 file changed, 35 insertions(+), 47 deletions(-)

(limited to 'views/default/tidypics')

diff --git a/views/default/tidypics/forms/edit.php b/views/default/tidypics/forms/edit.php
index d59d029cc..0ed8cffab 100644
--- a/views/default/tidypics/forms/edit.php
+++ b/views/default/tidypics/forms/edit.php
@@ -1,26 +1,24 @@
 <?php
 	/**
-	* Tidypics images edit/add page
+	* Tidypics images edit/add form
 	*  This form is used to:
 	*	- create albums
 	*	- edit albums
 	*	- edit images
 	*/
-	
+
 	//set stuff if we are editing existing album or image
 	if (isset($vars['entity'])) {
-		$title = sprintf(elgg_echo("album:edit"),$object->title);
 		$action = "tidypics/edit";
 		$title = $vars['entity']->title;
 		$body = $vars['entity']->description;
 		$tags = $vars['entity']->tags;
 		$access_id = $vars['entity']->access_id;
 		$subtype = $vars['subtype'];
-						
+
 	// if nothing is sent, create new, but only new albums are sent here
 	// new images are sent to upload.php
 	} else  {
-		$title = elgg_echo("album:add");
 		$action = "tidypics/addalbum";
 		$tags = "";
 		$title = "";
@@ -29,87 +27,77 @@
 			$access_id = ACCESS_DEFAULT;
 		else
 			$access_id = 0;
+		$subtype = 'album';
 	}
-			
-	//  in case we have some cached details
-	if (isset($vars['albumtitle'])) {
-		$title = $vars['albumtitle'];
-		$body = $vars['albumbody'];
-		$tags = $vars['albumtags'];
-	}
-        
-	$container_guid = get_input('container_guid');
-	if (!$container_guid) $container_guid = page_owner();
+
+	// group or individual 
+	$container_guid = page_owner();
 
 ?>
 <div class="contentWrapper">
 	<form action="<?php echo $vars['url']; ?>action/<?php echo $action; ?>" method="post">
 		<p>
 			<label><?php echo elgg_echo('album:title'); ?></label>
-			<?php  echo elgg_view("input/text", array("internalname" => "albumtitle", "value" => $title,));  ?>			
+			<?php  echo elgg_view("input/text", array("internalname" => "title", "value" => $title,));  ?>
 		</p>
-<?php 		
-		if ($vars['subtype'] == 'album' || $action == "tidypics/addalbum") {
+<?php
+		if ($subtype == 'album') {
 ?>
 		<p>
 			<label><?php echo elgg_echo('album:desc'); ?></label>
-			<?php  echo elgg_view("input/text",array("internalname" => "albumbody","value" => $body,)); ?>
+			<?php  echo elgg_view("input/text",array("internalname" => "descript","value" => $body,)); ?>
 		</p>
 <?php
 		} else {
 ?>
 		<p>
 			<label><?php echo elgg_echo('caption'); ?></label>
-			<?php  echo elgg_view("input/text",array("internalname" => "albumbody","value" => $body,)); ?>		
+			<?php  echo elgg_view("input/text",array("internalname" => "descript","value" => $body,)); ?>
 		</p>
 <?php
-		} //End album/image selection
+		} 
 ?>
 		<p>
 			<label><?php echo elgg_echo("tags"); ?></label>
-			<?php  echo elgg_view("input/tags", array( "internalname" => "albumtags","value" => $tags,));  ?>
+			<?php  echo elgg_view("input/tags", array( "internalname" => "tags","value" => $tags,));  ?>
 		</p>
 
-	<?php
-		
-		if($vars['subtype'] == 'image')
-		{		
-			//if this is an image -- 
-				//ALBUM COVER: ask to set image to album cover ::
-				//ACCESS:  only existing images come  here  :: so acess should already be initially set by album
-	
+<?php
+		if ($subtype == 'image') {
+			// should this album be the cover for the album
+			
+			// try to determine if it is already the cover - the pass of this variable doesn't work - leave for next version
 			$guid = $vars['entity']->guid;
 			$container_guid = $vars['entity']->container_guid;
 			$cover_guid = get_entity($container_guid)->cover;
-			
 			if($cover_guid == $vars['entity']->guid)
 				$cover = 'yes';
 			
-		?>
+?>
 		<p>
 			<label><?php echo elgg_echo("album:cover"); ?></label>
 			<?php  echo elgg_view("input/cover_checkbox", array( "internalname" => "cover", "value" => $cover, 'options' => array(elgg_echo('album:cover:yes'))));  ?>
 		</p>
-		<?php
-		}
-		else{
-			//this is a new or existing album, both of which access is editable
-		?>
+<?php
+		} else {
+			// album so display access control
+?>
 			<p>
-				<label><?php echo sprintf(elgg_echo('access'), "$subtype"); ?></label>
-				<?php echo elgg_view('input/access', array('internalname' => 'access_id','value' => $access_id)); ?>			
-			</p>		
-		
-		<?php
+				<label><?php echo elgg_echo('access'); ?></label>
+				<?php echo elgg_view('input/access', array('internalname' => 'access_id','value' => $access_id)); ?>
+			</p>
+
+<?php
 		}
 	
 		if (isset($vars['entity'])) {  
-		?>
-			<input type="hidden" name="albumpost" value="<?php echo $vars['entity']->getGUID(); ?>" />
-		<?php 
+?>
+			<input type="hidden" name="guid" value="<?php echo $vars['entity']->getGUID(); ?>" />
+<?php 
 		} 
-		?>
-		<input type="hidden" name="container_guid" value="<?php echo $container_guid; ?>" /> 		
+?>
+		<input type="hidden" name="container_guid" value="<?php echo $container_guid; ?>" />
+		<input type="hidden" name="subtype" value="<?php echo $subtype; ?>" />
 		<p><input type="submit" name="submit" value="<?php echo elgg_echo('save'); ?>" /></p>
 	</form>
 </div>
\ No newline at end of file
-- 
cgit v1.2.3