미스터S

영카트 5.0.3 함수 중복 및 ESCAPE 문제

· 2014-06-23 (월) 03:10:52 · 1077 · 2
/lib/common.lib.php
와
/adm/admin.lib.php

에서
member_delete() 함수가 중복되는걸로 확인됩니다.



shop_admin/orderformcartupdate.php
에서 "주문" 상태로 변경시
$_POST[ct_status] 가 넘어올때 ESCAPE_STRING 처리되어 한글이 깨져서 값변경이 안되는데
물론 config.php 에서 addslashes 처리하는 방법도 있겠지만

common.lib.php 의 get_search_string 함수가 보완된 함수인것 같은데 이곳저곳에 처리가 안되어 문제가 생기는곳이 보입니다.

shiop_admin/orderformcartupdate.php 의 상단에
$_POST[ct_status] = get_search_string($_POST['ct_status']);

처리를 해주어야 하지 않나 합니다.

또한 이 페이지에서 수량변경시 아래에 주문~품절 까지 상태변경 버튼을 눌러야 반영이 되는데
수량변경 확인에 대한 버튼은 안보입니다. 추가되어야할것 같습니다.

수량변경시 미수금액 확인이 필요하겠고 그 미수금액만큼 고객이 결제할 수 있는 방법이 필요하지 않나 합니다.
실제 쇼핑몰 운영해보면 고객들은 추가결제나 주문취소에 대한 번거로움을

오픈마켓과 비교하며 항의하는 경우가 많습니다.

상점운영 하는 실정에서 고객에게 "그럼 오픈마켓에서 구매해라" 라고 배짱부리기는 무리수입니다.
요즘 인터넷환경이 워낙 무섭다보니..

이런 사소한것들이 유료플러그인 등을 명분으로 개발사 참여 등으로 풍성해지고 개발사들도 이런것들로 수익을 얻을 수 있다면
참여하는 기업이나 프리랜서들이 많아 더욱 좋은 솔루션이 나올텐데.. 한번 고려해보심이..
|

댓글 2개

member_delete 함수는 common.lib.php 파일로 이전됐습니다. admin.lib.php 파일에서는 제거하지 않은신 것 같습니다.
한글이 깨지는 문제는 서버에 따라 mysql_real_escape_string 함수 결과물에서 한글이 깨지기 때문입니다.
이럴 경우 addslashes 함수를 대신 사용할 수 밖에 없습니다. get_search_string 함수는 검색에서 특수 문자를 제거하는
함수로 한글깨짐이 보안된 함수는 아닙니다. 수량 변경시 별도 확인 버튼은 필요없습니다. 상태 버튼 아무 거나 하나를
클릭하시면 바로 적용이 됩니다. 추후 확인 버튼 등을 추가할 계획은 없습니다. 필요하시다면 프로그램을 수정해 주시기 바랍니다.
추가 결제를 위한 방법으로 개인결제 기능을 제공하고 있습니다.

영카트5와 오픈마켓을 비교하는 것은 무리인 것 같습니다. 영카트5의 모든 소스는 공개되어 있으므로 필요하신 기능은
직접 개발하시거나 제작의뢰를 이용하시면 됩니다. 저희가 유료 프로그램을 개발하는 것은 영카트5의 무료화 정책과
받지 않기 때문에 유료로 프로그램을 개발해서 배포하는 것은 현재 고려하지 않고 있습니다.
음.. 뭔가 오해를 하신것 같습니다.

비교를 한것이 아니라.. 개발자들의 참여가있다면 더욱 좋은 프로그램이 만들어질것 같다는 의미입니다.
버튼역시 만들어달라는것이 아니고 타 사용자들을 위한 고려를 말씀드린 부분입니다.

감사합니다.
댓글을 작성하시려면 로그인이 필요합니다.

버그신고

4,191건

  문의게시판을 이용해 주세요 :) https://sir.kr/co_qa  

+
분류 제목 글쓴이 날짜
14-06-28
14-06-28
14-06-27
14-06-27
14-06-27
14-06-27
14-06-26
14-06-26
14-06-25
14-06-25
14-06-25
14-06-25
14-06-25
14-06-25
14-06-24
14-06-23
14-06-23
14-06-23
14-06-22
14-06-20