При удалении картинки, как передать часть url?

Question

[Антон Вебсайтовский]

Всем привет.

Сделал контроллер, в нем 3 актиона, каждый актион для своего шаблона.
В каждом есть загрузка фоток, грузятся фото все в одну таблицу, я не стал отдельные таблицы делать, а сделал просто по ид, 1, 2, 3, ну и каждый id подгружаю на своей нужной странице.

admin/prozhivanie/111 - допустим тут подгружаю информацию с таблицы под id 1.
admin/prozhivanie/222 - допустим тут подгружаю информацию с таблицы под id 2.
admin/prozhivanie/333 - допустим тут подгружаю информацию с таблицы под id 3.

Все работает отлично.

Каждую фотку можно отдельно удалять, я сделал актион для удаления, но есть загвоздка, при удалении, есть пере адресация, то есть удаляет фотку и возвращает на туже страницу где удалил фото, чтоб страница обновилась.
Проблема тут:
$this->redirect('admin/prozhivanie/111/' . $prozhivanies_id)
HTTP::Redirect('admin/prozhivanie/111/', $prozhivanies_id)
как мне передавать часть url где 111, чтоб скрипт понимал на какой странице находиться и возвращал туда же, если я нахожусь тут admin/prozhivanie/222 и нажимаю удалить, надо чтоб актион на удаление картинки это понимал и вместо 111, подставлял нужный урл.

А то приходиться делать три таких контроллера для каждой странице, вот подумал как-то же можно 1 сделать, вместо трех.

Код удаления картинки

public function action_delimg()
  {
    $id = (int)$this->request->param('id');
    $images = ORM::factory('Image', $id);
    $prozhivanies_id = $images->prozhivanie_id;

    if (!$images->loaded()) {
      HTTP::Redirect('admin/prozhivanie/111/', $prozhivanies_id);
    }

    $p_db = ORM::factory('Prozhivanie', $prozhivanies_id);
    if ($p_db->image_id == $id) {
      $p_db->image_id = 0;
      $p_db->save();
    }

    @unlink('media/uploads/prozhivanie/' . $images->image);
    @unlink('media/uploads/prozhivanie/small/small-' . $images->image);
    $images->delete();
    $this->redirect('admin/prozhivanie/111/' . $prozhivanies_id);
  }

В шаблоне

Тут разные формы и плюс загрузка фоток


Удаление картинки (часть кода).

<a href="/admin/prozhivanie/delimg/<?php echo $image->id ?>">Удалить</a>

Answer 1

[ThunderCat]

актион, prozhivanie, $prozhivanies... кошмарама...

Сделал контроллер, в нем 3 актиона, каждый актион для своего шаблона.

логичнее было бы задавать вьюшку (шаблон это обычно более общий вид, с заголовками, врапперами и тд) в зависимости от параметра, если экшны у вас одинаковые.

Comments

[Антон Вебсайтовский]

А почему кошмар, я записываю так чтоб понятно было ) латиница ) транслит.
Актионы да можно сказать одинаковые, разница только во вьюхе, что для каждого свою подгружаю, ну и для каждой вьюхи подгружаю из таблицы по иш нужную информацию.

Можно и перефразировать, для каждого актиона свой шаблон.
Но вопрос не в этом )

Я думаю проблему с удалением можно еще так решить, если сделать удаление через ajax, то есть без перезагрузки и тогда не надо урл подставлять нужный.

[Антон Вебсайтовский]

Я примерно тебя понимаю о чем говоришь, но пока не догоняю как сделать.

У меня для каждого актиона:

$content = View::factory('admin/prozhivanie/111')
    ->bind('id', $id)
      ->bind('prozhivanies', $prozhivanies)
      ->bind('data', $data);
$this->template->page_title = '111111111';
$this->template->content = $content;

Выходит можно сделать один актион, но в нем как-то подгружать к каждому урлу свой шаблон.

$content = View::factory('admin/prozhivanie/111')
    ->bind('id', $id)
      ->bind('prozhivanies', $prozhivanies)
      ->bind('data', $data);
$this->template->page_title = '1111111';
$this->template->content = $content;

    $content = View::factory('admin/prozhivanie/222')
    ->bind('id', $id)
      ->bind('prozhivanies', $prozhivanies)
      ->bind('data', $data);
$this->template->page_title = '2222222';
$this->template->content = $content;

    $content = View::factory('admin/prozhivanie/333')
    ->bind('id', $id)
      ->bind('prozhivanies', $prozhivanies)
      ->bind('data', $data);
$this->template->page_title = '3333333333';
$this->template->content = $content;

Но можно же и один сделать, а как тогда передавать титле для нужного урла и шаблон.

[ThunderCat]

Антон Вебсайтовский,

А почему кошмар, я записываю так чтоб понятно было ) латиница ) транслит.

Используйте нормальные англицкие слова, "тут так принято" ) и скилл поднимите в языке, и выглядеть солиднее будете в глазах коллег ).

Выходит можно сделать один актион, но в нем как-то подгружать к каждому урлу свой шаблон.

Да, так можно ) и $this->template->content = $content; достаточно добавить 1 раз в конце. А какой шаблон и заголовок определить в начале. Тогда все будет аккуратно и не будет дублироваться.

[Антон Вебсайтовский]

ThunderCat, понял, говоришь поверхностно. Видимо у самого опыт как у меня ) может чуть больше )
это выходит мне надо один актион сделать и называть его уже как положено action_index, а не так как я три сделал и каждый назвал под урл, ну и как-то надо сделать определение урла я так понимаю, чтоб передать на нужный свой вид шаблона.

[ThunderCat]

Антон Вебсайтовский,

понял, говоришь поверхностно. Видимо у самого опыт как у меня ) может чуть больше )

ну да, лет этак на 20 больше ). Говорю поверхностно по тому что в целом по уму вообще не так делают, но объяснять тебе в течение года как работать с фреймворками в зависимости от ситуации будет напряжно. Будет эффективнее помочь с направлением решения конкретно твоей кривоватой реализации коротко и по существу, а понимание правильного кода придет с опытом.

[Антон Вебсайтовский]

ThunderCat, по уму пример бы привел )

[ThunderCat]

Антон Вебсайтовский, пример чего? я даже не очень понимаю что вы строите, хотя бы написали что у вас происходит. Далее чисто догадки с вариантами...

3 актиона, каждый актион для своего шаблона

и далее 3 примерно одинаковых урл, отличающихся каким-то идентификатором... почему там 3 шаблона?

грузятся фото все в одну таблицу, я не стал отдельные таблицы делать, а сделал просто по ид, 1, 2, 3, ну и каждый id подгружаю на своей нужной странице.

Судя по всему это список фото, прикрепленные к разным сущностям, зачем им 3 разных шаблона - загадка, ведь фото оно и есть фото... непонятно короче. Это 3 разных сущности управляются? Тогда 3 разных экшна - это правильно и только так и должно быть, но урл тогда должен быть адекватным, например

admin/prozhivanie/villa/list 
admin/prozhivanie/villa/edit/5
admin/prozhivanie/home/list 
admin/prozhivanie/home/edit/3

ну или что там у вас еще есть. В роутинге у вас тогда admin - это апликэйшн бэкенд, prozhivanie - контроллер, villaEdit b viilaList - экшн, это в случае если вилла и хоум у вас разные сущности, а если это одна сущность с типом - тогда только list и edit экшны, или вообще другая структура, например отдельные контроллеры под виллу и дом, короче зависит от кучи вещей.

Answer 2

[Андрей Слащинин]

Попробуйте воспользоваться переменной $_SERVER['HTTP_REFERER'] — она хранит URL откуда пришел пользователь. Откуда пришел, пусть туда и возвращает.

Comments

[Антон Вебсайтовский]

а как передать его? мне конечно не весь урл нужен. часть.

[Андрей Слащинин]

Это глобальное значение, можешь проверить его внутри функции удаления, где ты делаешь redirect.

[Антон Вебсайтовский]

Андрей Слащинин, Типа так? $this->redirect('admin/prozhivanie/komfort/' . $_SERVER['HTTP_REFERER'] . $prozhivanies_id);

[Антон Вебсайтовский]

Проверил, да он так весь урл передает с https.
А если как-то по другому сделать, в актионе где-то вписать 111, и потом в шаблоне выводить, ну и передавать потом его. Вот только как, нет такого опыта, типа алиас.

[Антон Вебсайтовский]

Ну вообще так работает $this->redirect($_SERVER['HTTP_REFERER']); ))), все гениальное просто )

[Андрей Слащинин]

Отлично!

[Антон Вебсайтовский]

Андрей Слащинин, не знаю нормально ли так делать ), отдавать полный урл.

[Андрей Слащинин]

Антон Вебсайтовский, Можно предварительно его обработать и проверить сам факт наличия значения.