ifhub/ifhub.club
1
0
Fork 0
mirror of https://github.com/Oreolek/ifhub.club.git synced 2024-06-16 23:00:51 +03:00
Code Issues Releases Wiki Activity

Рефакторинг сообщений о системных ошибках + обновление framework

Browse source
This commit is contained in:
Mzhelskiy Maxim 2014-01-28 14:47:36 +07:00
parent 6185d99c9b
commit 9568c7e928
10 changed files with 119 additions and 191 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

119
application/classes/actions/ActionAjax.class.php
View file

@ -150,8 +150,7 @@ class ActionAjax extends Action {
* URL передали?
*/
if (!($sUrl=getRequestStr('url'))) {
$this->Message_AddError($this->Lang_Get('system_error'), $this->Lang_Get('error'));
return false;
return $this->EventErrorDebug();
}
/**
* Проверяем корректность target'а
@ -200,8 +199,7 @@ class ActionAjax extends Action {
$sName=getRequestStr('name');
$sValue=getRequestStr('value');
if (!in_array($sName,$aAllowData)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
$sId=getRequestStr('id');
if ($oMedia=$this->Media_GetMediaByIdAndUserId($sId,$this->oUserCurrent->getId())) {
@ -337,8 +335,7 @@ class ActionAjax extends Action {
* Файл был загружен?
*/
if (!isset($_FILES['filedata']['tmp_name'])) {
$this->Message_AddError($this->Lang_Get('system_error'), $this->Lang_Get('error'));
return false;
return $this->EventErrorDebug();
}
/**
* Проверяем корректность target'а
@ -350,14 +347,12 @@ class ActionAjax extends Action {
if ($sTargetId) {
$sTargetTmp=null;
if (!$this->Media_CheckTarget($sTargetType,$sTargetId)) {
$this->Message_AddError($this->Lang_Get('system_error'), $this->Lang_Get('error'));
return false;
return $this->EventErrorDebug();
}
} else {
$sTargetId=null;
if (!$sTargetTmp or !$this->Media_IsAllowTargetType($sTargetType)) {
$this->Message_AddError($this->Lang_Get('system_error'), $this->Lang_Get('error'));
return false;
return $this->EventErrorDebug();
}
}
@ -389,12 +384,10 @@ class ActionAjax extends Action {
* Если блог существует и он не персональный
*/
if (!is_string(getRequest('iBlogId'))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
if (!($oBlog=$this->Blog_GetBlogById(getRequest('iBlogId'))) or $oBlog->getType()=='personal') {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем локальный вьюер для рендеринга шаблона
@ -421,18 +414,15 @@ class ActionAjax extends Action {
*/
protected function EventVoteGetInfo() {
if ( ! is_string(getRequest('iTopicId')) ) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
if ( ! ($oTopic = $this->Topic_GetTopicById(getRequestStr('iTopicId', null, 'post'))) ) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'), $this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
if ( ! $oTopic->getVote() && ($this->oUserCurrent && $oTopic->getUserId() != $this->oUserCurrent->getId()) && (strtotime($oTopic->getDateAdd()) + Config::Get('acl.vote.topic.limit_time') > time())) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'), $this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
$oViewer = $this->Viewer_GetLocalViewer();
@ -456,8 +446,7 @@ class ActionAjax extends Action {
* Находим страну
*/
if (!($oCountry=$this->Geo_GetGeoObject('country',$iCountryId))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем список регионов
@ -488,8 +477,7 @@ class ActionAjax extends Action {
* Находим регион
*/
if (!($oRegion=$this->Geo_GetGeoObject('region',$iRegionId))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем города
@ -523,8 +511,7 @@ class ActionAjax extends Action {
* Комментарий существует?
*/
if (!($oComment=$this->Comment_GetCommentById(getRequestStr('idComment',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('comment_vote_error_noexists'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Голосует автор комментария?
@ -559,8 +546,7 @@ class ActionAjax extends Action {
*/
$iValue=getRequestStr('value',null,'post');
if (!in_array($iValue,array('1','-1'))) {
$this->Message_AddErrorSingle($this->Lang_Get('comment_vote_error_value'),$this->Lang_Get('attention'));
return;
return $this->EventErrorDebug();
}
/**
* Голосуем
@ -603,8 +589,7 @@ class ActionAjax extends Action {
* Топик существует?
*/
if (!($oTopic=$this->Topic_GetTopicById(getRequestStr('idTopic',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Голосует автор топика?
@ -632,8 +617,7 @@ class ActionAjax extends Action {
*/
$iValue=getRequestStr('value',null,'post');
if (!in_array($iValue,array('1','-1','0'))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('attention'));
return;
return $this->EventErrorDebug();
}
/**
* Права на голосование
@ -696,8 +680,7 @@ class ActionAjax extends Action {
* Блог существует?
*/
if (!($oBlog=$this->Blog_GetBlogById(getRequestStr('idBlog',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Голосует за свой блог?
@ -742,8 +725,7 @@ class ActionAjax extends Action {
return;
}
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('attention'));
return;
return $this->EventErrorDebug();
}
break;
case ModuleACL::CAN_VOTE_BLOG_ERROR_CLOSE:
@ -774,8 +756,7 @@ class ActionAjax extends Action {
* Пользователь существует?
*/
if (!($oUser=$this->User_GetUserById(getRequestStr('idUser',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Голосует за себя?
@ -803,8 +784,7 @@ class ActionAjax extends Action {
*/
$iValue=getRequestStr('value',null,'post');
if (!in_array($iValue,array('1','-1'))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('attention'));
return;
return $this->EventErrorDebug();
}
/**
* Голосуем
@ -854,15 +834,13 @@ class ActionAjax extends Action {
* Топик существует?
*/
if (!($oTopic=$this->Topic_GetTopicById($idTopic))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Тип топика - опрос?
*/
if ($oTopic->getType()!='question') {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Уже голосовал?
@ -876,8 +854,7 @@ class ActionAjax extends Action {
*/
$aAnswer=$oTopic->getQuestionAnswers();
if (!isset($aAnswer[$idAnswer]) and $idAnswer!=-1) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
if ($idAnswer==-1) {
@ -900,8 +877,7 @@ class ActionAjax extends Action {
$oViewer->Assign('oTopic',$oTopic);
$this->Viewer_AssignAjax('sText',$oViewer->Fetch("topics/poll_result.tpl"));
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
/**
@ -948,7 +924,7 @@ class ActionAjax extends Action {
$this->Favourite_UpdateFavourite($oFavourite);
return;
}
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return $this->EventErrorDebug();
}
/**
* Обработка избранного - топик
@ -967,15 +943,13 @@ class ActionAjax extends Action {
*/
$iType=getRequestStr('type',null,'post');
if (!in_array($iType,array('1','0'))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Топик существует?
*/
if (!($oTopic=$this->Topic_GetTopicById(getRequestStr('id',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Пропускаем топик из черновиков
@ -1003,8 +977,7 @@ class ActionAjax extends Action {
$this->Viewer_AssignAjax('bState',true);
$this->Viewer_AssignAjax('iCount', $oTopic->getCountFavourite());
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
if (!$oFavouriteTopic and !$iType) {
@ -1022,8 +995,7 @@ class ActionAjax extends Action {
$this->Viewer_AssignAjax('bState',false);
$this->Viewer_AssignAjax('iCount', $oTopic->getCountFavourite());
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
}
@ -1044,15 +1016,13 @@ class ActionAjax extends Action {
*/
$iType=getRequestStr('type',null,'post');
if (!in_array($iType,array('1','0'))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Комментарий существует?
*/
if (!($oComment=$this->Comment_GetCommentById(getRequestStr('id',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Комментарий уже в избранном?
@ -1073,8 +1043,7 @@ class ActionAjax extends Action {
$this->Viewer_AssignAjax('bState',true);
$this->Viewer_AssignAjax('iCount', $oComment->getCountFavourite());
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
if (!$oFavouriteComment and !$iType) {
@ -1092,8 +1061,7 @@ class ActionAjax extends Action {
$this->Viewer_AssignAjax('bState',false);
$this->Viewer_AssignAjax('iCount', $oComment->getCountFavourite());
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
}
@ -1114,15 +1082,13 @@ class ActionAjax extends Action {
*/
$iType=getRequestStr('type',null,'post');
if (!in_array($iType,array('1','0'))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Сообщение существует?
*/
if (!($oTalk=$this->Talk_GetTalkById(getRequestStr('id',null,'post')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Сообщение уже в избранном?
@ -1141,8 +1107,7 @@ class ActionAjax extends Action {
$this->Message_AddNoticeSingle($this->Lang_Get('talk_favourite_add_ok'),$this->Lang_Get('attention'));
$this->Viewer_AssignAjax('bState',true);
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
if (!$oFavouriteTalk and !$iType) {
@ -1158,8 +1123,7 @@ class ActionAjax extends Action {
$this->Message_AddNoticeSingle($this->Lang_Get('talk_favourite_del_ok'),$this->Lang_Get('attention'));
$this->Viewer_AssignAjax('bState',false);
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
@ -1274,8 +1238,7 @@ class ActionAjax extends Action {
*/
protected function EventBlogsGetByCategory() {
if (!($oCategory=$this->Blog_GetCategoryById(getRequestStr('id')))) {
$this->Message_AddError($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем все дочерние категории
@ -1401,8 +1364,7 @@ class ActionAjax extends Action {
*/
$idComment=getRequestStr('idComment',null,'post');
if (!($oComment=$this->Comment_GetCommentById($idComment))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Устанавливаем пометку о том, что комментарий удален
@ -1435,5 +1397,4 @@ class ActionAjax extends Action {
$this->Viewer_AssignAjax('bState',$bState);
$this->Viewer_AssignAjax('sTextToggle',$sTextToggle);
}
}
?>
}

45
application/classes/actions/ActionBlog.class.php
View file

@ -1283,15 +1283,13 @@ class ActionBlog extends Action {
*/
$idTopic=getRequestStr('idTarget',null,'post');
if (!($oTopic=$this->Topic_GetTopicById($idTopic))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Есть доступ к комментариям этого топика? Закрытый блог?
*/
if (!$this->ACL_IsAllowShowBlog($oTopic->getBlog(),$this->oUserCurrent)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
$idCommentLast=getRequestStr('idCommentLast',null,'post');
@ -1368,8 +1366,7 @@ class ActionBlog extends Action {
* Проверяем существование блога
*/
if(!$oBlog=$this->Blog_GetBlogById($sBlogId) or !is_string($sUsers)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Проверяем, имеет ли право текущий пользователь добавлять invite в blog
@ -1377,8 +1374,7 @@ class ActionBlog extends Action {
$oBlogUser=$this->Blog_GetBlogUserByBlogIdAndUserId($oBlog->getId(),$this->oUserCurrent->getId());
$bIsAdministratorBlog=$oBlogUser ? $oBlogUser->getIsAdministrator() : false;
if ($oBlog->getOwnerId()!=$this->oUserCurrent->getId() and !$this->oUserCurrent->isAdministrator() and !$bIsAdministratorBlog) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем список пользователей блога (любого статуса)
@ -1516,15 +1512,13 @@ class ActionBlog extends Action {
* Проверяем существование блога
*/
if(!$oBlog=$this->Blog_GetBlogById($sBlogId)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Пользователь существует и активен?
*/
if (!$oUser=$this->User_GetUserById($sUserId) or $oUser->getActivate()!=1) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Проверяем, имеет ли право текущий пользователь добавлять invite в blog
@ -1532,8 +1526,7 @@ class ActionBlog extends Action {
$oBlogUser=$this->Blog_GetBlogUserByBlogIdAndUserId($oBlog->getId(),$this->oUserCurrent->getId());
$bIsAdministratorBlog=$oBlogUser ? $oBlogUser->getIsAdministrator() : false;
if ($oBlog->getOwnerId()!=$this->oUserCurrent->getId() and !$this->oUserCurrent->isAdministrator() and !$bIsAdministratorBlog) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
$oBlogUser=$this->Blog_GetBlogUserByBlogIdAndUserId($oBlog->getId(),$oUser->getId());
@ -1541,7 +1534,7 @@ class ActionBlog extends Action {
$this->SendBlogInvite($oBlog,$oUser);
$this->Message_AddNoticeSingle($this->Lang_Get('blog.invite.notices.add',array('login'=>$oUser->getLogin())),$this->Lang_Get('attention'));
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return $this->EventErrorDebug();
}
}
/**
@ -1566,15 +1559,13 @@ class ActionBlog extends Action {
* Проверяем существование блога
*/
if(!$oBlog=$this->Blog_GetBlogById($sBlogId)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Пользователь существует и активен?
*/
if (!$oUser=$this->User_GetUserById($sUserId) or $oUser->getActivate()!=1) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Проверяем, имеет ли право текущий пользователь добавлять invite в blog
@ -1582,8 +1573,7 @@ class ActionBlog extends Action {
$oBlogUser=$this->Blog_GetBlogUserByBlogIdAndUserId($oBlog->getId(),$this->oUserCurrent->getId());
$bIsAdministratorBlog=$oBlogUser ? $oBlogUser->getIsAdministrator() : false;
if ($oBlog->getOwnerId()!=$this->oUserCurrent->getId() and !$this->oUserCurrent->isAdministrator() and !$bIsAdministratorBlog) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
$oBlogUser=$this->Blog_GetBlogUserByBlogIdAndUserId($oBlog->getId(),$oUser->getId());
@ -1594,7 +1584,7 @@ class ActionBlog extends Action {
$this->Blog_DeleteRelationBlogUser($oBlogUser);
$this->Message_AddNoticeSingle($this->Lang_Get('blog.invite.notices.remove',array('login'=>$oUser->getLogin())),$this->Lang_Get('attention'));
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return $this->EventErrorDebug();
}
}
/**
@ -1834,6 +1824,8 @@ class ActionBlog extends Action {
$sText = $this->Lang_Get('blog.personal_description');
}
$this->Viewer_AssignAjax('sText',$sText);
} else {
return $this->EventErrorDebug();
}
}
/**
@ -1857,8 +1849,7 @@ class ActionBlog extends Action {
*/
$idBlog=getRequestStr('idBlog',null,'post');
if (!($oBlog=$this->Blog_GetBlogById($idBlog))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Проверяем тип блога
@ -1934,8 +1925,7 @@ class ActionBlog extends Action {
*/
$this->Userfeed_unsubscribeUser($this->oUserCurrent->getId(), ModuleUserfeed::SUBSCRIBE_TYPE_BLOG, $oBlog->getId());
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
}
@ -1966,5 +1956,4 @@ class ActionBlog extends Action {
$this->Viewer_Assign('BLOG_USER_ROLE_REJECT', ModuleBlog::BLOG_USER_ROLE_REJECT);
$this->Viewer_Assign('BLOG_USER_ROLE_BAN', ModuleBlog::BLOG_USER_ROLE_BAN);
}
}
?>
}

13
application/classes/actions/ActionError.class.php
View file

@ -31,6 +31,12 @@ class ActionError extends Action {
'404' => array(
'header' => '404 Not Found',
),
'403' => array(
'header' => '403 Forbidden',
),
'500' => array(
'header' => '500 Internal Server Error',
),
);
/**
* Инициализация экшена
@ -64,7 +70,12 @@ class ActionError extends Action {
* Например, для 404 в хидере будет послан браузеру заголовок HTTP/1.1 404 Not Found
*/
if (array_key_exists($this->sCurrentEvent,$this->aHttpErrors)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error_'.$this->sCurrentEvent),$this->sCurrentEvent);
/**
* Смотрим есть ли сообщения об ошибках
*/
if (!$this->Message_GetError()) {
$this->Message_AddErrorSingle($this->Lang_Get('common.error.system.code.'.$this->sCurrentEvent),$this->sCurrentEvent);
}
$aHttpError=$this->aHttpErrors[$this->sCurrentEvent];
if (isset($aHttpError['header'])) {
$sProtocol=isset($_SERVER['SERVER_PROTOCOL']) ? $_SERVER['SERVER_PROTOCOL'] : 'HTTP/1.1';

3
application/classes/actions/ActionPeople.class.php
View file

@ -80,8 +80,7 @@ class ActionPeople extends Action {
if (is_string($sTitle) and mb_strlen($sTitle,'utf-8')) {
$sTitle=str_replace(array('_','%'),array('\_','\%'),$sTitle);
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
/**
* Как именно искать: совпадение в любой частилогина, или только начало или конец логина

51
application/classes/actions/ActionProfile.class.php
View file

@ -505,10 +505,10 @@ class ActionProfile extends Action {
* Пользователь авторизован?
*/
if (!$this->oUserCurrent) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
if (!$this->CheckUserProfile()) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Создаем запись
@ -561,16 +561,16 @@ class ActionProfile extends Action {
* Пользователь авторизован?
*/
if (!$this->oUserCurrent) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
if (!$this->CheckUserProfile()) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Получаем запись
*/
if (!($oWall=$this->Wall_GetWallById(getRequestStr('iId')))) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Если разрешено удаление - удаляем
@ -579,7 +579,7 @@ class ActionProfile extends Action {
$this->Wall_DeleteWall($oWall);
return;
}
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Ajax подгрузка сообщений стены
@ -590,7 +590,7 @@ class ActionProfile extends Action {
*/
$this->Viewer_SetResponseAjax('json');
if (!$this->CheckUserProfile()) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Формируем фильтр для запроса к БД
@ -626,10 +626,10 @@ class ActionProfile extends Action {
*/
$this->Viewer_SetResponseAjax('json');
if (!$this->CheckUserProfile()) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
if (!($oWall=$this->Wall_GetWallById(getRequestStr('iPid'))) or $oWall->getPid()) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Формируем фильтр для запроса к БД
@ -643,8 +643,7 @@ class ActionProfile extends Action {
} elseif (is_numeric(getRequest('iIdMore'))) {
$aFilter['id_more']=getRequest('iIdMore');
} else {
$this->Message_AddError($this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем сообщения и формируем ответ
@ -668,7 +667,7 @@ class ActionProfile extends Action {
* Пользователь авторизован?
*/
if (!$this->oUserCurrent) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
/**
* Создаем заметку и проводим валидацию
@ -701,14 +700,14 @@ class ActionProfile extends Action {
*/
$this->Viewer_SetResponseAjax('json');
if (!$this->oUserCurrent) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
if (!($oUserTarget=$this->User_GetUserById(getRequestStr('iUserId')))) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
if (!($oNote=$this->User_GetUserNote($oUserTarget->getId(),$this->oUserCurrent->getId()))) {
return parent::EventNotFound();
return $this->EventErrorDebug();
}
$this->User_DeleteUserNoteById($oNote->getId());
}
@ -907,19 +906,12 @@ class ActionProfile extends Action {
$this->Viewer_AssignAjax('sToggleText',$oViewerLocal->Fetch("actions/ActionProfile/friend_item.tpl"));
} else {
$this->Message_AddErrorSingle(
$this->Lang_Get('system_error'),
$this->Lang_Get('error')
);
return $this->EventErrorDebug();
}
return;
}
$this->Message_AddErrorSingle(
$this->Lang_Get('system_error'),
$this->Lang_Get('error')
);
return;
return $this->EventErrorDebug();
}
/**
* Отправляет пользователю Talk уведомление о принятии или отклонении его заявки
@ -1056,10 +1048,7 @@ class ActionProfile extends Action {
$this->Viewer_AssignAjax('sToggleText',$oViewerLocal->Fetch("actions/ActionProfile/friend_item.tpl"));
} else {
$this->Message_AddErrorSingle(
$this->Lang_Get('system_error'),
$this->Lang_Get('error')
);
return $this->EventErrorDebug();
}
return;
} else {
@ -1259,8 +1248,7 @@ class ActionProfile extends Action {
}
return;
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
}
/**
@ -1368,5 +1356,4 @@ class ActionProfile extends Action {
$this->Viewer_Assign('USER_FRIEND_REJECT',ModuleUser::USER_FRIEND_REJECT);
$this->Viewer_Assign('USER_FRIEND_DELETE',ModuleUser::USER_FRIEND_DELETE);
}
}
?>
}

17
application/classes/actions/ActionSettings.class.php
View file

@ -97,14 +97,14 @@ class ActionSettings extends Action {
$this->Viewer_SetResponseAjax('jsonIframe',false);
if(!isset($_FILES['foto']['tmp_name'])) {
return false;
return $this->EventErrorDebug();
}
/**
* Копируем загруженный файл
*/
$sFileTmp=Config::Get('sys.cache.dir').func_generator();
if (!move_uploaded_file($_FILES['foto']['tmp_name'],$sFileTmp)) {
return false;
return $this->EventErrorDebug();
}
/**
* Ресайзим и сохраняем именьшенную копию
@ -140,8 +140,7 @@ class ActionSettings extends Action {
$sFile=$this->Session_Get('sFotoFileTmp');
$sFilePreview=$this->Session_Get('sFotoFilePreviewTmp');
if (!file_exists($sFile)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
/**
* Определяем размер большого фото для подсчета множителя пропорции
@ -238,14 +237,14 @@ class ActionSettings extends Action {
$this->Viewer_SetResponseAjax('jsonIframe',false);
if(!isset($_FILES['avatar']['tmp_name'])) {
return false;
return $this->EventErrorDebug();
}
/**
* Копируем загруженный файл
*/
$sFileTmp=Config::Get('sys.cache.dir').func_generator();
if (!move_uploaded_file($_FILES['avatar']['tmp_name'],$sFileTmp)) {
return false;
return $this->EventErrorDebug();
}
/**
* Ресайзим и сохраняем уменьшенную копию
@ -275,8 +274,7 @@ class ActionSettings extends Action {
*/
$sFileAvatar=$this->Session_Get('sAvatarFileTmp');
if (!file_exists($sFileAvatar)) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем размер области из параметров
@ -724,5 +722,4 @@ class ActionSettings extends Action {
$this->Hook_Run('action_shutdown_settings');
}
}
?>
}

12
application/classes/actions/ActionSubscribe.class.php
View file

@ -101,8 +101,7 @@ class ActionSubscribe extends Action {
*/
$sTargetType=getRequestStr('target_type');
if (!$this->Subscribe_IsAllowTargetType($sTargetType)) {
$this->Message_AddError($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return ;
return $this->EventErrorDebug();
}
$sTargetId=getRequestStr('target_id') ? getRequestStr('target_id') : null;
$iValue=getRequest('value') ? 1 : 0;
@ -119,8 +118,7 @@ class ActionSubscribe extends Action {
* Проверка объекта подписки
*/
if (!$this->Subscribe_CheckTarget($sTargetType,$sTargetId,$iValue)) {
$this->Message_AddError($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return ;
return $this->EventErrorDebug();
}
/**
* Если подписка еще не существовала, то создаем её
@ -131,8 +129,6 @@ class ActionSubscribe extends Action {
$this->Message_AddNotice($this->Lang_Get('subscribe_change_ok'),$this->Lang_Get('attention'));
return ;
}
$this->Message_AddError($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return ;
return $this->EventErrorDebug();
}
}
?>
}

38
application/classes/actions/ActionTalk.class.php
View file

@ -551,12 +551,10 @@ class ActionTalk extends Action {
* Проверяем разговор
*/
if (!($oTalk=$this->Talk_GetTalkById(getRequestStr('idTarget')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
if (!($oTalkUser=$this->Talk_GetTalkUser($oTalk->getId(),$this->oUserCurrent->getId()))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Получаем комментарии
@ -614,12 +612,10 @@ class ActionTalk extends Action {
* Проверяем разговор
*/
if (!($oTalk=$this->Talk_GetTalkById(getRequestStr('cmt_target_id')))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
if (!($oTalkUser=$this->Talk_GetTalkUser($oTalk->getId(),$this->oUserCurrent->getId()))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Проверяем разрешено ли отправлять инбокс по времени
@ -641,8 +637,7 @@ class ActionTalk extends Action {
*/
$sParentId=(int)getRequest('reply');
if (!func_check($sParentId,'id')) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
$oCommentParent=null;
if ($sParentId!=0) {
@ -650,15 +645,13 @@ class ActionTalk extends Action {
* Проверяем существует ли комментарий на который отвечаем
*/
if (!($oCommentParent=$this->Comment_GetCommentById($sParentId))) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
/**
* Проверяем из одного топика ли новый коммент и тот на который отвечаем
*/
if ($oCommentParent->getTargetId()!=$oTalk->getId()) {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return;
return $this->EventErrorDebug();
}
} else {
/**
@ -714,7 +707,7 @@ class ActionTalk extends Action {
*/
$this->Talk_increaseCountCommentNew($oTalk->getId(),$oCommentNew->getUserId());
} else {
$this->Message_AddErrorSingle($this->Lang_Get('system_error'),$this->Lang_Get('error'));
return $this->EventErrorDebug();
}
}
/**
@ -861,11 +854,7 @@ class ActionTalk extends Action {
* Производим удаление пользователя из блекслиста
*/
if(!$this->Talk_DeleteUserFromBlacklist($idTarget,$this->oUserCurrent->getId())) {
$this->Message_AddErrorSingle(
$this->Lang_Get('system_error'),
$this->Lang_Get('error')
);
return;
return $this->EventErrorDebug();
}
$this->Message_AddNoticeSingle(
$this->Lang_Get(
@ -939,11 +928,7 @@ class ActionTalk extends Action {
* Удаляем пользователя из разговора, если удаление прошло неудачно - возвращаем системную ошибку
*/
if(!$this->Talk_DeleteTalkUserByArray($idTalk,$idTarget,ModuleTalk::TALK_USER_DELETE_BY_AUTHOR)) {
$this->Message_AddErrorSingle(
$this->Lang_Get('system_error'),
$this->Lang_Get('error')
);
return;
return $this->EventErrorDebug();
}
$this->Message_AddNoticeSingle(
$this->Lang_Get(
@ -1195,5 +1180,4 @@ class ActionTalk extends Action {
$this->Viewer_Assign('TALK_USER_DELETE_BY_SELF',ModuleTalk::TALK_USER_DELETE_BY_SELF);
$this->Viewer_Assign('TALK_USER_DELETE_BY_AUTHOR',ModuleTalk::TALK_USER_DELETE_BY_AUTHOR);
}
}
?>
}

10
application/frontend/i18n/ru.php
View file

@ -31,6 +31,13 @@ return array(
'empty' => 'Тут ничего нет',
'error' => array(
'save' => 'Ошибка сохранения',
'system' => array(
'code' => array(
'404' => 'К сожалению, такой страницы не существует. Вероятно, она была удалена с сервера, либо ее здесь никогда не было.',
'403' => 'Доступ к странице запрещен.',
'500' => 'Произошла внутренняя ошибка сервера.',
),
)
),
'success' => array(
'save' => 'Успешно сохранено',
@ -538,12 +545,10 @@ return array(
* Голосование за комментарий
*/
'comment_vote_error' => 'Попробуйте проголосовать позже',
'comment_vote_error_value' => 'Голосовать можно только +1 либо -1!',
'comment_vote_error_acl' => 'У вас не хватает рейтинга и силы для голосования!',
'comment_vote_error_already' => 'Вы уже голосовали за этот комментарий!',
'comment_vote_error_time' => 'Срок голосования за комментарий истёк!',
'comment_vote_error_self' => 'Вы не можете голосовать за свой комментарий!',
'comment_vote_error_noexists' => 'Вы голосуете за несуществующий комментарий!',
'comment_vote_ok' => 'Ваш голос учтен',
'comment_favourite_add' => 'добавить в избранное',
'comment_favourite_add_ok' => 'Комментарий добавлен в избранное',
@ -1290,7 +1295,6 @@ return array(
*/
'system_error_event_args' => 'Некорректное число аргументов при добавлении евента',
'system_error_event_method' => 'Добавляемый метод евента не найден',
'system_error_404' => 'К сожалению, такой страницы не существует. Вероятно, она была удалена с сервера, либо ее здесь никогда не было.',
'system_error_module' => 'Не найден класс модуля',
'system_error_module_no_method' => 'В модуле нет необходимого метода',
'system_error_cache_type' => 'Неверный тип кеширования',

2
framework

@ -1 +1 @@
Subproject commit dead633d0ac6e1738504bb03cce09c1c29fd0101
Subproject commit 4f4149c9b19fedb98e0d51cac1b5be587dcb16b1