Export to GitHub

xe-core - issue #3600

insertDocument 의 $obj 에서 eid 값으로도 확장변수 저장되도록 개선 요청 (패치제공)


Posted on Sep 29, 2013 by Helpful Elephant

document 모듈의 document.controller.php 의 function insertDocument($obj,...) 함수에서 확장변수를 $obj 로 전달하면 확장변수가 입력이 되는 걸로 알고 있습니다

XE 1.7 기준으로는 284 번째줄 ~ 301번째줄 까지인데

이 소스를 보면

우선순위로, 확장변수의 idx 값을 이용한 $obj->extra_vars1 값을 기준으로 하고 이게 없으면, 확장변수의 name 을 이용한 $obj->{extra_item->name} 을 기준으로 입력합니다.

idx 의 경우 확장변수 순서를 꼭 맞춰야한다는 한계가 있고 name 의 경우는 유사 성질이여도 게시판 성격에 따라 이름이 달라져야하는 경우가 있어서 전 eid 를 쓰는 경우가 많은데, 왜 eid 기준으로 하는건 없는지 모르겠네요

else if(isset($obj->{$extra_item->name})) $value = trim($obj->{$extra_item->name}); (295번째줄) 아래에

else if(isset($obj->{$extra_item->eid})) $value = trim($obj->{$extra_item->eid});

를 하나 넣어주면 좋을듯합니다.

eid 기준이 사이트를 만들다보면 훨씬 유용할때가 많기에 이 부분을 추가해주셨으면 좋겠습니다.

Comment #1

Posted on Nov 5, 2013 by Happy Hippo

https://github.com/xpressengine/core/issues/35

Status: MoveToGithub

Labels:
Type-Defect Priority-Medium