[PHP]Generowanie CSV i zapis do pliku - nie działa ;( Opcje

[phpamator]

Hej,
to znowu ja Wasz niepoprawny/niereformowalny phpamator (jak widać)

Napotkałem problem z którym nie potrafię sobie poradzić, może ktoś z was mógłby mnie oświecić.
Otóż pisząc mój kod który teoretycznie powinien działać (kod poniżej)

[HTML] pobierz, plaintext 
// sprawdzam czy są jakieś elementy zaznaczone, jeśli tak to lista "id" jest wysyłana do php - i to działa, dane są wysyłane
// lub jeśli nie zaznaczone wysyła tylko sygnał i wszystkie dane są wyrzucane do csv ale ...
$('#csvexport').on('click',function(){
			var selected = new Array();
			$('table input[type="checkbox"]:checked').each(function() {
				selected.push($(this).attr('id'));
			});
			if (selected.length > 0)
			{
				selected = selected ;
			}
			else
			{
				selected = null ;
			}
			event.preventDefault();
			$.ajax({
			  type: "POST",
			  url: "functions.php",
			  data:  {
				'action': 'csvexport',
				'userId': 1,
				'selected': selected
			  },
			  // dataType: "json",
			  // encode: true,
			}).success( function ( response ) {
			  if( response == 1 )
				{
					alert ( 'CSV Exported' ) ;
					// alert ( 'Export failed!' ) ;
					spinner() ;
				}
				spinner() ;
			});
		});
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
// .... ale mimo to nie dostaję nic spowrotem. spodziewam się że wyskoczy okienko i poprosi o wskazanie miejsca gdzie zapisać
// mój CSV
if (isset($_POST['action']) && $_POST['action'] == 'csvexport')
{
	if ( $_POST['selected'] == false ) 
	{
		$data = null ;
	}
	else
	{
		$data = $_POST['selected'] ;
	}
	$userId = $_POST[ 'userId' ] ;
	exportCsv( $userId, $data ) ; funkcja powinna zwrócić dane/plik
	ob_end_flush() ;
}
 
// pobieram dane do wygenerowania CSV
function getCsvData( $data )
{
	$link = getConfig() ;
	if ( !empty ( $data ) )
	{
		$clientData = $link->prepare("SELECT * FROM `mydb`.`mytable` WHERE `id` IN (".implode(',',$data).") ") ;
	}
	else
	{
		$clientData = $link->prepare("SELECT * FROM `mydb`.`mytable` ") ;
	}
	$clientData->execute() ;
	$result = $clientData->get_result() ;
	$assoc = $result->fetch_All(MYSQLI_ASSOC) ;
 	return  $assoc ;
}
 
// tworzę plik CSV
function exportCsv ( $userId, $data )
{
	$clients = getCsvData($data) ;
	ob_start() ;
	$output = fopen ( 'php://output', 'wb' ) ;
	fputcsv( $output, array_keys( $clients[0] ) ) ;
	foreach ( $clients as $client )
	{
		fputcsv( $output, $client ) ;
	}
	fclose( $output ) ;
	header ( 'Content-Type: text/csv; charset=utf-8' ) ;
	header ( 'Content-Disposition: attachment; filename=clients_report.csv' ) ;
	header ( 'Expires: 0' ) ;  
	header ( 'Cache-Control: no-cache' ) ;
	header ( 'Content-Length: '. ob_get_length() ) ;
	ob_end_flush() ;
}
 
 
[PHP] pobierz, plaintext 

wrzuciłem to do osobnego pliku i działa bez problemu

Co źle zrobiłem ?

[nospor]

Czy ja dobrze rozumiem ze slesz AJAX do pliku ktory wypluwa ci CSV z odpowiednimi headers i dziwisz sie ze sie nie schce sciagnac? No normalne. Poprostu przekieruj uztkownika do tego pliku a nie slesz tam AJAX.
Jak przekierowac? Normalnie,
document.location.href="adres do pliku"

[phpamator]

Teraz to ja nie rozumiem (IMG:style_emoticons/default/tongue.gif)
żeby rozjaśnić:

mam tabele gdzie wybieram które dane chcę mieć w CSV
więc zaznaczam i wysyłam ajaxem
dwie funkcje użyte w tym przykładzie pobieraj i generuj plik i zapisuj do bufora
i tu spodziewałem się mojego pliku poprzedzonego pytaniem "Gdzie chcesz zapisać wygenerowany plik"

Jak ty "nospor" byś to zrobił ?

[nospor]

ok, nie dolukalem ze przesylasz parametry jeszcze
Tylko ze parametry nadal mozesz przeslac do pliku czyli nadal jak pisalem wczesniej


document.location.href="http://url_do_sciagniecia_pliku?tutaj_parametry"


Jesli jednak nadal sie upierasz przy ajax, to url na ktory idzie ajax ma nie zwracac pliku z naglowkami, tylko sama zawartosc pliku, ajax ja pobierzez i potem w js przygotowujesz zwrotke do pobrania pliku dla usera. W necie masz masze przykladow jak zrobic download w js

[Salvation]

Ja bym zrobił to inaczej. Plik wygenerował i zapisał po stronie PHP. Zwrócił na front jedynie zahashowaną nazwę pliku i front zrobiłby przekierowanie na ten właśnie plik do pobrania.

[phpamator]

Ja bym zrobił to inaczej. Plik wygenerował i zapisał po stronie PHP. Zwrócił na front jedynie zahashowaną nazwę pliku i front zrobiłby przekierowanie na ten właśnie plik do pobrania.

Mógł byś podeprzeć to jakimś przykładem ?

[nospor]

@Salvation skoro i tak musisz w twoim przypadku i tak we frontend przekierowac do pliku do sciagniecia, to czemu nie lepiej przekierowac do pliku ktory generuje co ma generowac? Wydaje sie to byc zbedna dodatkowa funkcjonalnoscia

[Salvation]

Kod masz już prawie gotowy. Wystarczy zmienić funkcję `exportCsv` na taką by zapisywała plik - na przykład z użyciem natywnych funkcji: https://www.php.net/manual/en/function.fputcsv.php lub z użyciem jakiejś bliblioteki. Jak Ci wygodniej.

Zamiast statycznej nazwy pliku: file.csv generuj ją sobie np. w ten sposób:

[PHP] pobierz, plaintext 
$filenameHash = md5(sprintf('csvexport_%s_%s', $userId, time()));
$filename = sprintf('%s.csv', $filenameHash);
[PHP] pobierz, plaintext 

I zwracaj albo sam hash ($filenameHash), albo nazwę z rozszerzeniem: $filename.

I w tym ifie co masz, to ostatnie dwie linijki zamieniasz na to:

[PHP] pobierz, plaintext 
echo exportCsv($userId, $data);
[PHP] pobierz, plaintext 

Dodaj jeszcze tylko typowanie argumentów i typ zwrotek funkcji i będzie dobrze.

@Salvation skoro i tak musisz w twoim przypadku i tak we frontend przekierowac do pliku do sciagniecia, to czemu nie lepiej przekierowac do pliku ktory generuje co ma generowac? Wydaje sie to byc zbedna dodatkowa funkcjonalnoscia

Tak, w sumie to masz rację z tym. Tylko nie wiem czy użycie przekierowania "jawnego" (bezpośredniego) nie zaburzy założeń biznesowych. Może jest jakiś powód do tego, że to leci po AJAX-ie.