Cześć,

Używam API napisane w PHP do generowania dokumentów (System generuje mi zestawienie sprzedaży dla danego klienta w docx). Niestety mój developer który mi to zrobił kilka lat temu nie jest już zainteresowany inwestowaniem swojego czasu potrzebuję pomocy. Sam ogarniam podstawy i potrafię wykonać większość modyfikacji jakie potrzebuję, ale niestety z tym nie potrafię sobie poradzić.

System zapisuje wygenerowane pliki na serwerze. Mój developer na pewnym etapie prac dodatkowo załatał jeszcze jedną dziurę bezpieczeństwa, czyli pobieranie wygenerowanych dokumentów docx bezpośrednio przez serwer http, czyli bez potrzeby logowania. Logowanie obejmuje tylko dostęp do php, pliki statyczne w tym dokumenty docx są serwowana bezpośredni przez serwer http z pominięciem php. Przerobił to tak, żeby te pliki także szły przez php.

W tej chwili jak wykomentuje to o co mi chodzi pliki zapisują się w jednej lokalizacji documents/. Chiciałem zrobić, żeby każdy agent miał pliki w swoim katalogu czyli documents/PiotrP drugi documents/MarcinS itd.
Niestety ale mam problem. Nie wiem czy to co wrzucę jest wystarczające no ale spróbuję opisać do czego doszedłem:

plik DocumentGenerate.php

[PHP] pobierz, plaintext 
//zdefiniowanie scieżki zapisu dokumentu dla każdego agenta
if ($agent == 'Piotr aaa') {
    $outputFilePath = DOCUMENT_PATH . '/PiotrP/' . $outputFileName;
} elseif ($agent == 'Marcin bbb') {
    $outputFilePath = DOCUMENT_PATH . '/MarcinS/' . $outputFileName;
} elseif ($agent == 'Michal ccc') {
    $outputFilePath = DOCUMENT_PATH . '/MichalT/' . $outputFileName;
} else {
    $outputFilePath = DOCUMENT_PATH . '/' . $outputFileName;
}
 
$TBS->Show(OPENTBS_FILE, $outputFilePath);
$TBSError = trim(ob_get_clean());
if (!is_file($outputFilePath)) {
    throw new Exception ('Document has not been generated <br/>' . $TBSError);
}
 
echo json_encode(array(
    'status' => 'OK',
    'html' => array(
        array(
            'content' => '<div class="container alert alert-success text-center">
                            <a target="_blank" href="/getDocument.php?name=' . $outputFileName . '">' . $outputFileName . '</a>
                           </div>',
            'container' => HtmlConfig::CONTAINER_DOCUMENT_LINK,
            'pasteType' => 'replace'),
    )
));
[PHP] pobierz, plaintext 

Dzięki temu generuje się link do pliku do pobrania. Plik generuje się prawidłowo w odpowiedniej lokalizacji dla tych trzech użytkowników. Dodam że oryginalnie nie było tych if /else if i wszystko zapisywało sie w katalogu głównym documents/

Plik drugi getDocument.php (chyba tutaj jest jakiś problem) odpowiada za możliwość pobrania tych plików. Obecnie wygląda tak:

[PHP] pobierz, plaintext 
<?php
 
include_once 'DocumentDeclaration.php';
include_once 'DocumentData.php';
include_once 'AppConfig.php';
 
if (!array_key_exists('name', $_GET) || !is_string($_GET['name'])) {
    throw new Exception('File name is not determined.');
}
 
$fileName = $_GET['name'];
 
if (false !== strpos($fileName, '/')) {
    throw new Exception('Incorrect file path.');
}
 
//zapis orygianlny
//$filePath = DOCUMENT_PATH . '/' . $_GET['name'];
//
//Tutaj modyfikacja skad pobierać plik - nie działa if (brak zmiennej $agent chyba)
if ($agent == 'Piotr aaa') {
    $filePath = DOCUMENT_PATH . '/PiotrP/' . $_GET['name'];
} elseif ($agent == 'Marcin bbb') {
    $filePath = DOCUMENT_PATH . '/MarcinS/' . $_GET['name'];
} elseif ($agent == 'Michal ccc') {
    $filePath = DOCUMENT_PATH . '/MichalT/' . $_GET['name'];
} else {
    $filePath = DOCUMENT_PATH . '/' . $_GET['name'];
}
 
 
if (!is_file($filePath)) {
    //throw new Exception('File does not exist!');
    throw new Exception(print_r($filePath, 1) );
}
 
if (!is_readable($filePath)) {
    throw new Exception('File is not readable.');
}
 
header('Content-Description: File Transfer');
header('Content-Type: application/vnd.openxmlformats-officedocument.wordprocessingml.document');
header("Content-Disposition: attachment; filename=" . $fileName);
header('Content-Transfer-Encoding: binary');
header('Expires: 0');
header('Cache-Control: must-revalidate');
header('Pragma: public');
header('Content-Length: ' . filesize($filePath));
readfile($filePath);
[PHP] pobierz, plaintext 

No i niestety ale nie mogę pobrać tych wygenerownych plików. Po kliknięciu na link zamiast okna do zapisu pliku dostaję komunikat:

{"status":"ERR","message":[{"body":"C:\\wamp\\www\\api\\document\/XXX_Zbigniew_20190424_170454_other_en.docx","type":"danger"}]}

Tak jakby nie działały te if w drugim pliku.

Przykład który pokaże może gdzie problem.
Powiedzmy, że generuję plik dla agenta Piotr aaa. Plik zapisuje się prawidłowo w katalogu documents/PiotrP. Przy tym co wyżej, nie mogę pobrać pliku (komunkiat o błędzie). Ale jak dla testów dopiszę w drugim pliku w tej linijce:

[PHP] pobierz, plaintext 
else {
    $filePath = DOCUMENT_PATH . '/' . $_GET['name'];
}
[PHP] pobierz, plaintext 

lokalizajcę gdzie zapisał sie pliki czyli ;

[PHP] pobierz, plaintext 
else {
    $filePath = DOCUMENT_PATH . '/PiotrP/' . $_GET['name'];
}
[PHP] pobierz, plaintext 

to wszystko działa prawidłowo i w momencie kliknięcie linku otwiera okno z zapisem pliku.

Mam nadzieję że to co napisąłem ma jakiś sens. System jest dość rozbudowany i składa się z 30-40 plików ale oczywiście większość myślę że jest tutaj bez znaczenia. Drugi plik podałem w całości. Pierwszy tylko fragment ale tam myśle ze wszystko działa bo zapisuje prawidłowo.
Dziękuję za wskazówki / podpowiedzi

A skad sie bierze zmienna $agent? Nigdzie nie widze jej tworzenia

ps: pozatym w druim pliku zjadles ELSEIF i dales samo IF. Przejrzyj pierwszy plik gdzie masz ELSEIF i skopiuj dokladnie bo jak nic to tez jest problememem

Dzięki za zainteresowanie.
Tak jak pisałem system jest dość rozbudowany (i pewnie dlatego sobie nie radzę).

Jest plik o nazwię DocumentData.php w którym m.in. jest:

[PHP] pobierz, plaintext 
public function getData($final = false) {
        $this->captureLeadId();
        $this->captureContactId();
        $this->captureDocumentParameters();
        $this->capturePrimaryPerson();
        $this->capturePriority();
        $this->capturePersonWithDefaultOrder();
        $this->captureAgent();
        $this->captureAddress();
        $this->captureLeadSource();
        $this->captureConsider();
        $this->captureRecommend();
        $this->captureChildren();
        $this->capturePolicies();
        $this->captureDataDependOn();
        if ($final) {
            $this->captureFields();
            //throw new Exception('<pre>' . var_export($this->data, 1) . '</pre><br>');
        }
[PHP] pobierz, plaintext 

a chwile niżej w tym samym pliku jest:

[PHP] pobierz, plaintext 
private function captureAgent() {
        $this->data['agent'] = $_POST['agent'];
        $agentFull = new AgentFull();
        $this->data['agentFull'] = $agentFull->getByAgentName($_POST['agent']);
        $this->data['agentPhone'] = $_POST['agentPhone'];
        $this->data['broker'] = $_POST['broker'];
        $brokerAddress = new BrokerAddress();
        $this->data['brokerAddress'] = $brokerAddress->getByBrokerName($_POST['broker']);
        $brokerBackground = new BrokerBackground();
        $this->data['brokerBackground'] = $brokerBackground->getByBrokerName($_POST['broker']);
        $agentSignature = new AgentSignature();
        $this->data['agentSignature'] = $agentSignature->getByAgentName($_POST['agent']);
    }
[PHP] pobierz, plaintext 

Tak myślałem na początku że nie przechodzi ta zmienna więc dla pewności dodałem na poczatku

[PHP] pobierz, plaintext 
include_once 'DocumentData.php';
[PHP] pobierz, plaintext 

w pliku getDocument.php dla tego pliku

Sprawdz moj PS z poprzedniego posta bo tam narozrabiales

Ok. Widziałem to elseif. Chyba kiedyś kombinowałem i tak zostało. teraz poprawiłem i jest tak jak w pierwszym poście z elseif. Dalej nie działa.

Daj w takim razie przed blokiem IF
var_dump($agent);

a po bloku IF
var_dump($filePath);
i pokaza jakie wartosci zwrocilo

null
string 'C:\wamp\www\apiRightone\document/XXX_Zbigniew_20190424_175408_other_en.docx' (length=76)

Czyli jak wdzisz $agent jest null wiec co ci nie wyszlo jego dodanie do drugiego pliku

ps: samo include_once 'DocumentData.php'; raczej nie wywola funkcji tam zawartych. Przesledz lepiej pierwszy plik jak on odpala to wszystko wczesniej

Tutaj cały plik pierwszy czyli documentGenerate.php

[PHP] pobierz, plaintext 
<?php
/**
 * Created by PhpStorm.
 * User: herb
 * Date: 25.04.15
 * Time: 19:37
 */
 
function base64ToPng( $base64_string, $output_file ) {
    $ifp = fopen( $output_file, "wb" );
    fwrite( $ifp, base64_decode( $base64_string) );
    fclose( $ifp );
}
 
include_once 'DocumentDeclaration.php';
include_once 'DocumentData.php';
include_once 'AdditionalDataWriter.php';
include_once 'AppConfig.php';
 
$documentData = new DocumentData();
$documentData = $documentData->getData(true);
 
//tu sa wszystkie dane ze strony ze wszytskich formularzy
// wiec w tym miejscu zapiszemy dane do bazy
$writer = new AdditionalDataWriter();
$writer->writeData($documentData);
 
$declaration = new DocumentDeclaration();
$fields = $declaration->getFieldsDefinition($documentData['type'], $documentData['language'], $documentData['dual']);
$declaration = $declaration->getDeclarationByTypeAndLanguage($documentData['type'], $documentData['language'], $documentData['dual']);
 
$primarySignatureImagePath = CLIENT_SIGNATURE_PATH . DIRECTORY_SEPARATOR . date('Y-m-d') . '_' . $documentData['primary']['firstName'] . '_' . $documentData['primary']['lastName']
    . '_' . $declaration['template'] .'_primary.png';
$secondarySignatureImagePath = CLIENT_SIGNATURE_PATH . DIRECTORY_SEPARATOR . date('Y-m-d') . '_' . $documentData['primary']['firstName'] . '_' . $documentData['primary']['lastName']
    . '_' . $declaration['template'] .'_secondary.png';
 
base64ToPng( $_POST['signature']['primary']['data'] , $primarySignatureImagePath);
base64ToPng( $_POST['signature']['secondary']['data'] , $secondarySignatureImagePath);
 
$signatureUsed = $_POST['signatureUsed'];
 
include_once 'tbs_class.php';
include_once 'tbs_plugin_opentbs.php';
 
$TBS = new clsTinyButStrong;
$TBS->Plugin(TBS_INSTALL, OPENTBS_PLUGIN);
$TBS->SetOption(array('noerr' => true));
 
extract($documentData);
//extract($policies);
//extract($policiesLevel);
//extract($policiesMark);
//extract($policiesLevelMark);
 
 
ob_start();
$TBS->LoadTemplate($declaration['templatePath'], OPENTBS_ALREADY_UTF8);
 
foreach ($policies as $policeKey => $police) {
    // z uwagi na błąd w TBS - nie wykożystywać jednego alisau bloku dwa razy, dodawane są aliasy z kolejnymi liczbami
    $TBS->MergeBlock($policeKey . ',' . $policeKey . '2,' . $policeKey. '3,' . $policeKey . '4', $police);
}
 
// Bloki dla dokumentu common, strony w tym dokumencie zależą od formylarza
 
//throw new Exception('<pre>' . print_r($documentData, 1) . '</pre>');
/*foreach ($documentData['f'] as $sectionName => $sectionData) {
    // jeśli w danych z formularzy jest sekcja,
    // której nazwa ZACZYNA SIĘ od przedrostaka dla dokumentu commo
    // to dodaj blok o pełnej nazwie sekcji do TBS
    if ($commonPrefix == substr($sectionName, 0, strlen($commonPrefix))) {
        $TBS->MergeBlock($sectionName, $sectionData);
        //throw new Exception('<pre>' . print_r($sectionName, 1) . '</pre>');
        //break;
    }
}*/
 
if ($documentData['f']['comCoAL']['selected'] == 'yes') {
    $TBS->MergeBlock('comCoAL', $documentData['f']);
} else {
    $TBS->MergeBlock('comCoAL', []);
}
 
if ($documentData['f']['comCL']['selected'] == 'yes') {
    $TBS->MergeBlock('comCL', $documentData['f']);
} else {
    $TBS->MergeBlock('comCL', []);
}
if ($documentData['f']['comCH']['selected'] == 'yes') {
    $TBS->MergeBlock('comCH', $documentData['f']);
} else {
    $TBS->MergeBlock('comCH', []);
}
 
//funkcja usuwa puste klucze tablicy
function array_filter_recursive($data) { 
    foreach ($data as &$value) { 
        if (is_array($value)) { 
            $value = array_filter_recursive($value); 
        } 
    } 
return array_filter($data); 
} 
$arraynoempty = array_filter_recursive($documentData['f']);
//$TBS->MergeBlock('existlife', $arraynoempty['exist']['life']);
//throw new Exception('<pre>' . print_r($arraynoempty['exist']['life'], 1) . '</pre>');
 
//usuniecie pierwszego klucza dla tablicy. Dodano warunek gdyby niezaznaczono warosci hold w formularzu.
function cut_array($input, $data) {
    if ($input == 'yes' || $input == 'no' || $input == 'Not disclosed') {
	$cutArray = array_slice($data, 1); //usun pierwsza tablice gdy zaznaczono pole hold
    }
    else {
	$cutArray = $data;	//nie wycinaj pierwszej tablicy gdy pola hold zapomniał zaznaczyć użytkownik formularza
    }
return $cutArray;
}
//wykonaj cut_array dla klucza [exist][life] tablicy z usuniętymi pustymi kluczami i zmiennymi
$cutArray = cut_array($documentData['f']['exist']['life']['0']['hold'], $arraynoempty['exist']['life']);
$TBS->MergeBlock('existlife', $cutArray); //wrzuć dane tablicy do blocku existlife
//throw new Exception('<pre>' . print_r($cutArray, 1) . '</pre>');
 
//throw new Exception('<pre>' . print_r($arraynoempty['existincomeprot'], 1) . '</pre>');
//throw new Exception('<pre>' . print_r($documentData['f']['existincomeprot'], 1) . '</pre>');
//throw new Exception('<pre>' . print_r($documentData['f'], 1) . '</pre>');
//throw new Exception('<pre>' . print_r($_POST, 1) . '</pre>');
//
$TBS->MergeBlock('b,b1,b2', $documentData['f']); //wrzucenie danych o formularzu w block b, b1 i b2
$TBS->MergeBlock('address,address2', $documentData['address']); //wrzucenie danych adresowych w block address
$TBS->MergeBlock('brokeraddress, brokeraddress2', $documentData['brokerAddress']); //wrzucenie danych adresowych w block address
$TBS->MergeBlock('companyaddress,companyaddress2', $documentData['policies']['Group']['0']['DOC']['companyaddress']); //wrzucenie danych adresowych w block address
 
$outputFileName =  $documentData['primary']['lastName'] . '_' . $documentData['primary']['firstName'] . '_' . date('Ymd_His') . '_' . $declaration['template'];
 
//zdefiniowanie scieżki zapisu dokumentu dla każdego agenta
if ($agent == 'Piotr aaa') {
    $outputFilePath = DOCUMENT_PATH . '/PiotrP/' . $outputFileName;
} elseif ($agent == 'Marcin bbb') {
    $outputFilePath = DOCUMENT_PATH . '/MarcinS/' . $outputFileName;
} elseif ($agent == 'Michal ccc') {
    $outputFilePath = DOCUMENT_PATH . '/MichalT/' . $outputFileName;
} else {
    $outputFilePath = DOCUMENT_PATH . '/' . $outputFileName;
}
 
$TBS->Show(OPENTBS_FILE, $outputFilePath);
$TBSError = trim(ob_get_clean());
if (!is_file($outputFilePath)) {
    throw new Exception ('Document has not been generated <br/>' . $TBSError);
}
 
echo json_encode(array(
    'status' => 'OK',
    'html' => array(
        array(
            'content' => '<div class="container alert alert-success text-center">
                            <a target="_blank" href="/getDocument.php?name=' . $outputFileName . '">' . $outputFileName . '</a>
                           </div>',
            'container' => HtmlConfig::CONTAINER_DOCUMENT_LINK,
            'pasteType' => 'replace'),
    )
));
[PHP] pobierz, plaintext 

No i powtorz wszystko w pliku drugim co w pliku pierwszym ma zwiazek z $documentData

tak wygląda plik DocumentData.php

[PHP] pobierz, plaintext 
<?php
/**
 * Created by PhpStorm.
 * User: herb
 * Date: 25.04.15
 * Time: 20:25
 */
 
include_once 'DependOnCompanyDefault.php';
include_once 'DependOnCompanySpecified.php';
include_once 'PolicyParameterTranslator.php';
include_once 'PolicyParameterAdder.php';
include_once 'AgentSignature.php';
include_once 'Broker.php';
include_once 'Agent.php';
include_once 'DocumentDeclaration.php';
include_once 'PolicyGroup.php';
 
class DocumentData {
 
    private $data = array();
    private $primary = '';
    private $translator;
    private $adder;
 
    public function __construct() {
        $this->translator = new PolicyParameterTranslator();
        $this->adder = new PolicyParameterAdder();
    }
 
    public function getData($final = false) {
        $this->captureLeadId();
        $this->captureContactId();
        $this->captureDocumentParameters();
        $this->capturePrimaryPerson();
        $this->capturePriority();
        $this->capturePersonWithDefaultOrder();
        $this->captureAgent();
        $this->captureAddress();
        $this->captureLeadSource();
        $this->captureConsider();
        $this->captureRecommend();
        $this->captureChildren();
        $this->capturePolicies();
        $this->captureDataDependOn();
        if ($final) {
            $this->captureFields();
            //throw new Exception('<pre>' . var_export($this->data, 1) . '</pre><br>');
        }
 
        //throw new Exception('<pre>' . print_r($this->data, 1) . '</pre><br><pre>' . print_r($_POST, 1) . '</pre>');
        //throw new Exception('<pre>' . var_export($this->data, 1) . '</pre><br>');
        //throw new Exception('<pre>' . var_export($_POST, 1) . '</pre><br>');
        //throw new Exception('<pre>' . print_r($_POST, 1) . '</pre>');
        //throw new Exception('<pre>' . print_r($this->data, 1));
        return $this->data;
    }
 
    private function captureLeadId()
    {
        $this->data['leadid'] = $_POST['leadid'];
    }
    private function captureContactId()
    {
        $this->data['contactid'] = $_POST['contactid'];
    }
 
    private function captureDocumentParameters() {
        if (!array_key_exists('person', $_POST)) {
            throw new Exception('Lack of clients data');
        }
 
        if (!array_key_exists('documentType', $_POST)) {
            throw new Exception('Lack of documentType parameter.');
        }
 
        if (!array_key_exists('language', $_POST)) {
            throw new Exception('Lack of language parameter.');
        }
 
        $this->data['type'] = $_POST['documentType'];
        $this->data['language'] = $_POST['language'];
        $this->data['dual'] = true;
        if (array_key_exists('onePerson', $_POST)) {
            $this->data['dual'] = false;
        }
    }
 
    private function capturePrimaryPerson() {
        if (!array_key_exists('primary', $_POST['person']) || !in_array($_POST['person']['primary'], array('A', 'B'))) {
            throw new Exception('Pleas, mark one person as primary.');
        }
 
        $this->primary = $_POST['person']['primary'];
    }
 
    private function capturePriority() {
        $this->data[$this->prescribePriority('A')] = $_POST['person']['A'];
        $this->data[$this->prescribePriority('B')] = $_POST['person']['B'];
    }
 
    private function prescribePriority($person) {
        if ($this->primary == $person) {
            return 'primary';
        }
 
       return 'secondary';
    }
 
    private function capturePersonWithDefaultOrder() {
        $this->data['person'] = [
            'A' => $_POST['person']['A'],
            'B' => $_POST['person']['B']
        ];
    }
 
    private function captureAgent() {
        $this->data['agent'] = $_POST['agent'];
        $agentFull = new AgentFull();
        $this->data['agentFull'] = $agentFull->getByAgentName($_POST['agent']);
        $this->data['agentPhone'] = $_POST['agentPhone'];
        $this->data['broker'] = $_POST['broker'];
        $brokerAddress = new BrokerAddress();
        $this->data['brokerAddress'] = $brokerAddress->getByBrokerName($_POST['broker']);
        $brokerBackground = new BrokerBackground();
        $this->data['brokerBackground'] = $brokerBackground->getByBrokerName($_POST['broker']);
        $agentSignature = new AgentSignature();
        $this->data['agentSignature'] = $agentSignature->getByAgentName($_POST['agent']);
    }
...
...
[PHP] pobierz, plaintext 

jak w pliku getDocument.php powtórzę ten zapis (czyli pobierz wszystkie zmienne)

[PHP] pobierz, plaintext 
$documentData = new DocumentData();
$documentData = $documentData->getData(true);
[PHP] pobierz, plaintext 

to dostaje

[HTML] pobierz, plaintext 
{"status":"ERR","message":[{"body":"Lack of clients data","type":"danger"}]}
[HTML] pobierz, plaintext 

Zakładam że powinienem odwołać się do funkcji cuptureAgent tylko jak to zrobić?

??wesoła twórczość

??koniec twórczości??

Zdaje sie dosc wyraznie napisalem ze masz powtorzyc caly kod z pliku pierwszego, ktory dotyczy zmiennej $documentData. Czego nie rozumiesz w tym zdaniu?
Odpal sobie w edytorze CTRL+F, wpisz $documentData i to ci znajdzie wszystkie jego wystapienia. Masz je powtorzyc w pliku drugim.

Hej,

Już pisałem, że jak powtórzę wszystkie wpisy gdzie jest $documentData to dostaję błąd:

{"status":"ERR","message":[{"body":"Lack of clients data","type":"danger"}]}

Ten błąd wynika z części kodu jaki jest w pliku DocumentData.php, który już wkleiłem, żeby pokazać skąd się bierze:

[PHP] pobierz, plaintext 
private function captureDocumentParameters() {
        if (!array_key_exists('person', $_POST)) {
            throw new Exception('Lack of clients data');
        }
 
        if (!array_key_exists('documentType', $_POST)) {
            throw new Exception('Lack of documentType parameter.');
        }
 
        if (!array_key_exists('language', $_POST)) {
            throw new Exception('Lack of language parameter.');
        }
 
        $this->data['type'] = $_POST['documentType'];
        $this->data['language'] = $_POST['language'];
        $this->data['dual'] = true;
        if (array_key_exists('onePerson', $_POST)) {
            $this->data['dual'] = false;
        }
    }
[PHP] pobierz, plaintext 

Ponownie napisze, że wykonywanie tego tutaj jest zbędne bo to odpowiada za walidację parametrów takich jak: Czy istnieje klient, czy istnieje typ polisy, czy wybrano język). Innymi słowy zbędne tutaj. Ja z całego tego pliku potrzebuję uzyskać jedną zmienną jaką jest $agent.

Tak naprawdę to ten błąd dostaję już jak powtórzę:

[PHP] pobierz, plaintext 
include_once 'DocumentData.php';
 
$documentData = new DocumentData();
$documentData = $documentData->getData(true);
[PHP] pobierz, plaintext 

Poza tym nie ma sensu powtarzać części z nich bo dotyczą zupełnie innych kwestii i są kompletnie nie związane ze zmienną agent.

Moje ograniczone myślenie prowadzi mnie do tego, żeby dodać nową funkcje publiczną w pliku DocumentData.php:

[PHP] pobierz, plaintext 
 //nowa funkcja publiczna dla potrzeb pliku getDocument.php
    public function getAgent() {
        $this->captureAgent();
        return $this->data['agent'];
//        return $this->data;
    }
[PHP] pobierz, plaintext 

i wtedy w getDocument.php

[PHP] pobierz, plaintext 
//wywołanie funkcji publicznej getAgent utworzonej dla tych potrzeb w DocumentData.php
include_once 'DocumentData.php';
 
$documentData = new DocumentData;
$agent = $documentData->getAgent();
[PHP] pobierz, plaintext 

No ale cały czas jak wrzucę poniżej:

[PHP] pobierz, plaintext 
var_dump($agent);
[PHP] pobierz, plaintext 

To po klinknięciu na link do ściągania pliku dostaję
'null'

Jak zmienię mój wpis w DocumentData.php na:

[PHP] pobierz, plaintext 
 //nowa funkcja publiczna dla potrzeb pliku getDocument.php
    public function getAgent() {
        $this->captureAgent();
//      return $this->data['agent'];
        return $this->data;
    }
[PHP] pobierz, plaintext 

to dostaję:
array (size=7)
'agent' => null
'agentFull' => null
'agentPhone' => null
'broker' => null
'brokerAddress' => null
'brokerBackground' => null
'agentSignature' => null

Jest to oczywiście związane z tym że tak naprawdę nie wywołuję mojej nowej funkcji publicznej getAgent na poziomie plików określających typ dokumentu jaki ma być wygenerowany. Wieć albo musiąłbym to dodać w tych plikach (ich stała cześć skaładowa wyglada tak:

[PHP] pobierz, plaintext 
include_once 'DocumentData.php';
include_once 'AppConfig.php';
 
class FormOther {
    private $groups = array (
    );
 
    public function __construct(DocumentData $documentData = null) {
 
        if (!$documentData) {
            $documentData = new DocumentData();
        }
 
        $documentData = $documentData->getData();
 
        //budowanie zmiennych do dokumentu w oparciu o wybrane wartości tutaj
 
    }
 
    public function getGroups() {
        return $this->groups;
    }
} 
[PHP] pobierz, plaintext 

Albo tak jak pisałem wyłuskać z tej istniejącej funkcji publicznej getData tylko zmienne dla funkcii prywatnej captureAgent. Niestety tego nie potrafię.

No dobra, moj blad, spojrzalem tylko na kod ktory wkleiles a tam mi brakowalo tego i owego.
Ok, teraz widze ze agen idzie poprostu z POSTa. A z tego co widze do pliku dobierasz sie linkiem wiec raz ze postem nic nie wysylasz a dwa ze i tak nie ma tam agenta.

Jak nie zalezy ci na bezpieczenstwie, tudziez i tak twoja aplikacja za bardzo bezpieczna nie jest - patrzac w jaki sposob teraz leci agent to mozna odniesc takie wlasnie wrazenie - to dodaj tego agenta poprostu do linku i pobieraj go z GET i juz
Jesli chcesz to zrobic choc odrobine ladniej, to trzymaja agenta w SESSION i przy pobieraniu pliku pobieraj go wlasnie stamtad

Hej,

dlaczego nie mogę dobrać się do tej zmiennej skoro ona jest dostępne np jak zrobię

[PHP] pobierz, plaintext 
throw new Exception('<pre>' . print_r($this->data, 1));
[PHP] pobierz, plaintext 

Na poziomie tej funkcji publicznej public function getData w pliku DocumentData.php

[XML] pobierz, plaintext 
Array
(
    [leadid] => 
    [contactid] => 624745000001111111
    [type] => Other
    [language] => PL
    [dual] => 1
 
    [person] => Array
        (
            [A] => Array
                (
                    [sex] => male
                    [firstName] => Zxxxx
                    [lastName] => XXX
                    [dob] => 1958-01-01
                    [age] => xx years
                    [mobile] => 01---
                    [email] => 
                    [marital] => M
                    [smoking] => No
                    [occupation] => Ccccc
                    [company] => A---
                    [monthlyIncome] => 
                    [socialIncome] => 
                )
 
            [B] => Array
                (
                    [sex] => female
                    [firstName] => Jxxxxx
                    [lastName] => xxx
                    [dob] => 1960-01-01
                    [age] => xx years
                    [mobile] => 01----
                    [email] => 
                    [marital] => M
                    [smoking] => N
                    [occupation] => Sxxxx
                    [company] => Gxxx
                    [monthlyIncome] => 
                    [socialIncome] => 
                )
 
        )
 
    [agent] => Piotr aaa
    [agentFull] => Piotr aaa
    [agentPhone] => 01 xxxxxxxx
    [broker] => Piotr aaa
    [brokerAddress] => Array
...
...
...
[XML] pobierz, plaintext 

Na koniec czy dodać tą zmienną do SESSION powinienem na poziomie pliku documentGenerate.php (bo tam mam tą zmienną).

Ja nie wiem, ja widze, ze w DocumetnData agent idzie stad
$this->data['agent'] = $_POST['agent'];
czyli z posta.

Plik pobierasz na podstawie zwyklego linku z tego co widzialem tak? Czyli POSTa nie wysylasz a tym samym nie masz agent w POST. Czy moze sie myle i plik pobierasz inaczej?


Czyli w akcji pobierania pliku masz tego agent? No to nie rozumiem w czym problem

Ok. Puściłem zmienną agent w linku w pliku documentGenerate.php

[PHP] pobierz, plaintext 
 <a target="_blank" href="/getDocument.php?name=' . $outputFileName . '&agent=' .$agent .'">' . $outputFileName . '</a>
[PHP] pobierz, plaintext 

w w pliku getDocument.php zczytałem

[PHP] pobierz, plaintext 
$agent = $_GET['agent'];
[PHP] pobierz, plaintext 

I działa. Dziękuję tobie za pomoc, cierpliwość i czas jaki poświęciłeś.