[Symfony2][SF2][SF]Obsługa wyjątku przy usuwaniu rekordu z bazy Opcje

[amii]

W ajaxie obsługuje usuwanie rekordów z bazy i w przypadku błędu (np. usuwania rekordu z powiązanym kluczem obcym) mam zwróconą stronę z błędem 500 (widzę to z poziomu konsoli).
To co chcę osiągnąć to:
- przechwycenie tego wyjątku
- zwrócenie zamiast strony błędu, odpowiedzi w formacie json

W kontrolerze mam tak ale cały czas zwraca stronę z błędem 500

[PHP] pobierz, plaintext 
 public function usunKlientaAction()
    {
		$em = $this->getDoctrine()->getManager();
                $response = new JsonResponse();
		$request = $this->container->get('request');        
		$klient_id = $request->request->get('klient');
 
		$repository = $this->getDoctrine()->getRepository('MojeBundleCMSBundle:Klienci');
		$ka = $repository->find($klient_id);
 
		try {
			$em->remove($ka);
			$em->flush();
		}
		catch (Exception $e) {
			$response->setData(array('class' => 'error', 'message' => 'Wystapił błąd podczas usuwania rekordu!'));
			return $response;
		}
 
		$response->setData(array('class' => 'notice', 'message' => 'Rekord został poprawnie usuniety!'));
		return $response;
    }
[PHP] pobierz, plaintext 

[Damonsson]

A co Ty chcesz przechwytywać? Skoro masz zawsze 500, to ślesz request pod zły adres, a nie masz żadnego błędu w zapytaniu, bo nawet tam nie dochodzisz. Więc po prostu to popraw.

Ten post edytował Damonsson 16.07.2014, 14:26:20

[amii]

Mam błąd np. kiedy usuwam rekord z kluczem obcym. I wtedy pluje coś w stylu: 'cascading remove : integrity constraint violation 1451' chodzi mi o to żeby użytkownik dostał info jako json a nie żeby ajax zwracał error 500

[Damonsson]

No to pokaż tego AJAXa którego wysyłasz.

[ohm]

To może lepiej sprawdzić dlaczego wywala 500? A nie próbować go catchować. Relacje masz dobrze "połapane"?
No i czy musisz rekord fizycznie usuwać? Czy nie wygodniej dać mu jakiś status i ustawiać np na deleted.

Ten post edytował ohm 16.07.2014, 14:51:18

[amii]

AJAX w php wywoływany z poziomu JS to jest właśnie ten fragment co zamieściłem czyli metoda usunKlientaAction()

Jeśli zaś chodzi o JS, to po prostu odwołanie się do tej metody

[JAVASCRIPT] pobierz, plaintext 
$( document ).ready(function() {
		  (function( $ ) {
				$('.remove').click(function() {
					if (confirm('Czy na pewno chcesz usunąć tego klienta ?')) {
						$.ajax({
							url : '{{path("ajax_update_klienciusun")}}',
							type: 'POST',
							data : {
								'klient' : $(this).attr('data-attr')
							},
							cache : false,
							success : function(data) {
								$("#flash-messanger").addClass(data.class).html(data.message);
								$("#flash-messanger").fadeIn(800);
								setTimeout(function() {
									$("#flash-messanger").fadeOut('fast');
									window.location.reload(true);
								}, 5000); // <-- hide after 5000 milliseconds
							}
						});
					}
				});
			}(jQuery));
		});
[JAVASCRIPT] pobierz, plaintext 

[Damonsson]

[JAVASCRIPT] pobierz, plaintext 
$( document ).ready(function() {
		  (function( $ ) {
				$('.remove').click(function() {
					if (confirm('Czy na pewno chcesz usunąć tego klienta ?')) {
						$.ajax({
							url : '{{path("ajax_update_klienciusun")}}',
							type: 'POST',
							data : {
								'klient' : $(this).attr('data-attr')
							},
							cache : false,
							success : function(data) {
								$("#flash-messanger").addClass(data.class).html(data.message);
								$("#flash-messanger").fadeIn(800);
								setTimeout(function() {
									$("#flash-messanger").fadeOut('fast');
									window.location.reload(true);
								}, 5000); // <-- hide after 5000 milliseconds
							},
    statusCode: {
        500: function() {
          alert("500");
        }
      }
						});
					}
				});
			}(jQuery));
		});
[JAVASCRIPT] pobierz, plaintext 

Wyłapało? Oprócz Success też masz możliwość dodania error jeszcze

[Crozin]

@Damonsson: 500 Internal Server Error oznacza błąd w skrypcie (nieprzechwycony wyjątek w tym przypadku). Chyba pomyliłeś z 404 Not Found.
@amii: Popraw aplikację, bo zakładasz, że jej działanie oparte jest na błędzie. W zależności od tego co dokładnie potrzebujesz:
1. Albo sprawdź czy dany obiekt w ogóle da się usunąć w danym momencie i na podstawie tego zdecyduj czy w ogóle pokazać linka do usunięcia czy nie. Oczywiście w samej akcji usuwającej również powinieneś mieć tego typu zabezpieczenie.
2. Albo popraw swój kod tak, by w ogóle działał, tj. przed usunięciem danego klienta usuń wszystkie dane które mają z nim relację. Tutaj @ohm również dobrze zauważył, że podobnie jak w większości przypadków tak i tutaj dużo lepiej jest oznaczyć dane jako usunięte - usuwanie danych jest niebezpieczne.

PS. Obiekt Requesta powinieneś przekazać jako argument akcji; nie masz żadnego zabezpieczenia przed CSRF, a co za tym idzie Twój kod jest mało bezpieczny.

[Damonsson]

Dobrze prawisz, myślałem o 500, a napisałem o 404

Ten post edytował Damonsson 16.07.2014, 16:36:23

[amii]

Crozin - poprawiłem ten błąd z usuwaniem i kluczami obcymi poprzez nadanie nullable=true, onDelete="SET NULL" dla tabeli z kluczami obcymi.