Skip to content
This repository has been archived by the owner on Jul 29, 2020. It is now read-only.

Práce s externími identifikátory #16

Open
RAD-X opened this issue Jan 23, 2019 · 3 comments
Open

Práce s externími identifikátory #16

RAD-X opened this issue Jan 23, 2019 · 3 comments
Assignees

Comments

@RAD-X
Copy link

RAD-X commented Jan 23, 2019

Tento kód sice funguje (kupodivu - haluz vs úmysl? :-) ) ale háže warning pro rawurlencode protože je předána array namisto stringu.

$save = array(
	'id' => array(
		$addr->id, // timto identifikujeme co se ma upravovat
		// pridany externi identifikator 
		$external_id
	),
	'eanKod' => $member_id
);
$adresar = new \FlexiPeeHP\Adresar($save, $this->make_conf());
$savedata = $adresar->insertToFlexiBee();

samozřejmě já to řešil raději takto

$ids = array(
	$addr->id, // timto identifikujeme co se ma upravovat
	// pridany externi identifikator 
	$external_id
);
$ids = json_encode($ids);
$save = array(
	'id' => $ids,
	'eanKod' => $member_id
);
$adresar = new \FlexiPeeHP\Adresar($save, $this->make_conf());
$savedata = $adresar->insertToFlexiBee();

Chvíli jsem nad tím musel přemýšlet jak to správně vložit protože jsem nenašel žádnou dokumentaci ani příklad a metoda/funce setMyKey ve FlexiBeeRO evidentně není určená k přiřazování externích identifikátorů - resp. je nefunkční v mém pojetí.
Nebo je na toto nějaký jiný přístup?

Jinak díky za tuto library.

@RAD-X
Copy link
Author

RAD-X commented Jan 23, 2019

Ještě jsem si našel chvilku a vidím že jsem přehlédl ve FlexibeeRW funkce changeExternalID a addExternalID sic je to oproti mému příkladu o jedno API volání navíc, ale metody tu na to jsou. 😃

Takže vlastně k úplné dokonalosti by bylo fajn automaticky skládat/encodovat položky parametrů pokud nejsou typu string tak aby nevznikalo PHP varování které je zmíněné v mém předešlém příspěvku.

@Vitexus
Copy link
Collaborator

Vitexus commented Jan 23, 2019

Ano, tak jak navrhujete v předchozím příspěvku jsem chtěl k externím id přistupovat ale nakonec mě nic nedonutiolo tento mechanizmus implementovat.

@RAD-X RAD-X closed this as completed Jan 27, 2019
@Vitexus
Copy link
Collaborator

Vitexus commented Jan 29, 2019

Zatím jsem tedy podniknul první krok k tomu aby bylo možné co navrhujete:
a6a6985

@Vitexus Vitexus reopened this Feb 10, 2019
@Vitexus Vitexus self-assigned this May 30, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants