Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ClientStatus kontrolü hakkında #17

Open
1YalcinSahin1 opened this issue Aug 19, 2020 · 2 comments
Open

ClientStatus kontrolü hakkında #17

1YalcinSahin1 opened this issue Aug 19, 2020 · 2 comments

Comments

@1YalcinSahin1
Copy link

1YalcinSahin1 commented Aug 19, 2020

Merhaba,

Herhangi bir service istek attığımda BaseCall __construct() girdiğinden dolayı sürekli client açıyor ?

getCities() isteği için de client açıyor , getTopLevelCategories() için de.

Yaptığınız ClientStatus mantığını anlayamadım.

Ayrıca editörler city , category gibi ifadeleri(magic method algılamıyor).

/**
 * Class N11ClienClass
 * @property-read CategoryService $category
 * @property-read ProductService $product
 * @property-read CityService $city
 * @property-read ShipmentCompanyService $shipmentcompany;
 * @property-read ShipmentService $shipment;
 * @property-read ProductSellingService $selling
 * @property-read ProductStockService $stock
 * @property-read OrderService $order
 * @property-read WebHookService $webhook
 * @package App\Classes\N11Service
 */
class N11ClienClass extends N11Client
{
}

Clientleri bu şekilde bir class üzerinden açarlarsa sorunları da düzelir.

Edit: Sorun PHPDoc verilmesinden dolayı oluşuyor.Metotların bazıları 500 hatasına dönüyor.

$client parametresi geçirdiğiniz fonksiyonlarda $client = null yapıp kalıtım yaparsam sorun çözülüyor. Ama parametre olarak $client vermenizi anlamadım

@ismail0234
Copy link
Owner

Mimariyi ilk tasarladığımda o şekilde kurgulamıştım. Daha sonradan sürekli açılan clienti sabit tek bir client'e indirecektim ama fazla vaktim olmadığı için sadece düşünce olarak kaldı. Parametre olarak verilen $client her oturum için iç kodda kullanılması için veriliyor. Client static ile tek bir bağlantı haline gelirse onlara gerek kalmadan sorununuz çözülür ama şuan pek müsait olamadığım için bunu yapamıyorum.

@1YalcinSahin1
Copy link
Author

1YalcinSahin1 commented Aug 21, 2020 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants