fixed member UX/UI

git-svn-id: http://xe-core.googlecode.com/svn/branches/1.5.0@9021 201d5d3c-b55e-5fd7-737f-ddc643e51545

modules/member/member.admin.controller.php

@@ -57,6 +57,20 @@
             $this->add('member_srl', $args->member_srl);
             $this->setMessage($msg_code);
 
+			$profile_image = $_FILES['profile_image'];
+			if (is_uploaded_file($profile_image['tmp_name'])){
+				$oMemberController->insertProfileImage($args->member_srl, $profile_image['tmp_name']);
+			}
+
+			$image_mark = $_FILES['image_mark'];
+			if (is_uploaded_file($image_mark['tmp_name'])){
+				$oMemberController->insertImageMark($args->member_srl, $image_mark['tmp_name']);
+			}
+
+			$image_name = $_FILES['image_name'];
+			if (is_uploaded_file($image_name['tmp_name'])){
+				$oMemberController->insertImageName($args->member_srl, $image_name['tmp_name']);
+			}
 			if(!in_array(Context::getRequestMethod(),array('XMLRPC','JSON'))) {
 				$returnUrl = Context::get('success_return_url') ? Context::get('success_return_url') : getNotEncodedUrl('', 'module', 'admin', 'act', 'dispMemberAdminList');
 				header('location:'.$returnUrl);

modules/member/member.admin.view.php

@@ -233,22 +233,25 @@
 					}
 					$list_order[] = $signupItem;
 				}
-				foreach($extendItems as $form_srl=>$item_info){
+
-					unset($signupItem);
+				if (is_array($extendItems)){
-					$signupItem->name = $item_info->column_name;
+					foreach($extendItems as $form_srl=>$item_info){
-					$signupItem->title = $item_info->column_title;
+						unset($signupItem);
-					$signupItem->type = $item_info->column_type;
+						$signupItem->name = $item_info->column_name;
-					$signupItem->member_join_form_srl = $form_srl;
+						$signupItem->title = $item_info->column_title;
-					$signupItem->mustRequired = in_array($key, $mustRequireds);
+						$signupItem->type = $item_info->column_type;
-					$signupItem->required = ($item_info->required == 'Y');
+						$signupItem->member_join_form_srl = $form_srl;
-					$signupItem->isUse = ($item_info->is_active == 'Y');
+						$signupItem->mustRequired = in_array($key, $mustRequireds);
-					$signupItem->description = $item_info->description;
+						$signupItem->required = ($item_info->required == 'Y');
-					if ($signupItem->imageType){
+						$signupItem->isUse = ($item_info->is_active == 'Y');
-						$signupItem->max_width = $config->{$key.'_max_width'};
+						$signupItem->description = $item_info->description;
-						$signupItem->max_height = $config->{$key.'_max_height'};
+						if ($signupItem->imageType){
+							$signupItem->max_width = $config->{$key.'_max_width'};
+							$signupItem->max_height = $config->{$key.'_max_height'};
+						}
+						$list_order[] = $signupItem;
 					}
-					$list_order[] = $signupItem;
+				}
- 				}
 				$member_config->signupForm = $list_order;
 			}
 
@@ -273,7 +276,8 @@
 							$functionName = 'doDeleteImageMark';
 						}
 						if($target->src){
-							$inputTag = sprintf('<p class="a"><img src="%s" alt="%s" width="80" height="80" /> <button type="button" class="text" onclick="%s(%d);return false;">%s</button></p>'
+							$inputTag = sprintf('<p class="a"><span id="%s"><img src="%s" alt="%s" /> <button type="button" class="text" onclick="%s(%d);return false;">%s</button></span></p>'
+												,$formInfo->name.'tag'
 												,$target->src
 												,$formInfo->title
 												,$functionName
@@ -381,7 +385,7 @@
 						<div class="krZip">
 							<div class="a" id="zone_address_search_%s" %s>
 								<label for="krzip_address1_%s">%s</label><br />
-								<input type="text" id="krzip_address1_%s" value="" />
+								<input type="text" id="krzip_address1_%s" value="%s" />
 								<button type="button">%s</button>
 							</div>
 							<div class="a" id="zone_address_list_%s" style="display:none">
@@ -397,9 +401,9 @@
 						<script type="text/javascript">jQuery(function($){ $.krzip('%s') });</script>
 EOD;
 						$inputTag = sprintf($content 
-											,$extendForm->column_name,  $extendForm->value[0]?'style="display:none"':''
+											,$extendForm->column_name,  $extendForm->value[0]?'':'style="display:none"'
 											,$extendForm->column_name,  $lang->msg_kr_address
-											,$extendForm->column_name
+											,$extendForm->column_name, $extendForm->value[0]
 											,$lang->cmd_search
 											,$extendForm->column_name
 											,$extendForm->column_name, $extendForm->column_name

modules/member/tpl/insert_member.html

@@ -1,11 +1,16 @@
 <load target="js/member_admin.js" />
 <load target="js/krzip_search.js" />
 <!--%load_js_plugin("ui.datepicker")-->
+<script type="text/javascript" >
+	xe.lang.deleteProfileImage = '{$lang->msg_delete_extend_form}';
+	xe.lang.deleteImageMark = '{$lang->msg_delete_extend_form}';
+	xe.lang.deleteImageName = '{$lang->msg_delete_extend_form}';
+</script>
 <div cond="$XE_VALIDATOR_MESSAGE" class="message {$XE_VALIDATOR_MESSAGE_TYPE}">
 	<p>{$XE_VALIDATOR_MESSAGE}</p>
 </div>
 <div class="content" id="content">
-	<form action="./" class="form" ruleset="@insertMember" method="post">
+	<form action="./" class="form" ruleset="@insertMember" method="post" enctype="multipart/form-data">
 		<input type="hidden" name="act" value="procMemberAdminInsert" />
 		<input type="hidden" name="member_srl" value="{$member_srl}" />
 		<input type="hidden" name="signature" value="{htmlspecialchars($member_info->signature)}" />

modules/member/tpl/js/member_admin.js

@@ -248,21 +248,43 @@ function completeSearchKrZip(ret_obj, response_tags, callback_args) {
 
 /* 프로필 이미지, 이미지 이름, 마크 삭제 */
 function doDeleteProfileImage(member_srl) {
-    var fo_obj = get_by_id("fo_image");
+	if (!member_srl) return;
-    fo_obj.member_srl.value = member_srl;
+
-    procFilter(fo_obj, delete_profile_image);
+	if (!confirm(xe.lang.deleteProfileImage)) return false;
+
+	exec_xml(
+		'member',
+		'procMemberDeleteProfileImage',
+		{member_srl:member_srl},
+		function(){jQuery('#profile_imagetag').remove()},
+		['error','message','tpl']
+	);
 }
 
 function doDeleteImageName(member_srl) {
-    var fo_obj = get_by_id("fo_image");
+	if (!member_srl) return;
-    fo_obj.member_srl.value = member_srl;
+
-    procFilter(fo_obj, delete_image_name);
+	if (!confirm(xe.lang.deleteImageName)) return false;
+	exec_xml(
+		'member',
+		'procMemberDeleteImageName',
+		{member_srl:member_srl},
+		function(){jQuery('#image_nametag').remove()},
+		['error','message','tpl']
+	);
 }
 
 function doDeleteImageMark(member_srl) {
-    var fo_obj = get_by_id("fo_image");
+	if (!member_srl) return;
-    fo_obj.member_srl.value = member_srl;
+
-    procFilter(fo_obj, delete_image_mark);
+	if (!confirm(xe.lang.deleteImageMark)) return false;
+	exec_xml(
+		'member',
+		'procMemberDeleteImageMark',
+		{member_srl:member_srl},
+		function(){jQuery('#image_marktag').remove()},
+		['error','message','tpl']
+	);
 }
 
 

modules/member/tpl/member_info.html

@@ -9,13 +9,13 @@
 {@$value = $lang->find_account_question_items[$value].' : '.$memberInfo['find_account_answer']}
 			<!--@elseif($formInfo->name == 'profile_image' && $memberInfo['profile_image'])-->
 {@$target = $memberInfo['profile_image']}
-{@$value = '<img src="{$target->src}" width="80" height="80" />'}
+{@$value = '<img src="'.$target->src.'" />'}
 			<!--@elseif($formInfo->name == 'image_name' && $memberInfo['image_name'])-->
 {@$target = $memberInfo['image_name']}
-{@$value = '<img src="{$target->src}" width="80" height="80" />'}
+{@$value = '<img src="'.$target->src.'" />'}
 			<!--@elseif($formInfo->name == 'image_mark' && $memberInfo['image_mark'])-->
 {@$target = $memberInfo['image_mark']}
-{@$value = '<img src="{$target->src}" width="80" height="80" />'}
+{@$value = '<img src="'.$target->src.'" />'}
 			<!--@end-->
 		<!--@else-->
 {@$title = $extend_form_list[$formInfo->member_join_form_srl]->column_title}