movemoveapp/laravel-dadata

PHP SDK Laravel пакет работы с сервисом DaData.ru, для исправления синтаксических ошибок в информации контактных данных клиентов сайта и вывода подсказок поля форм.

laravel-dadata's Issues

The package breaks the Russian localization for Carbon

When installing the package, the Russian localization of Carbon breaks down.

'locale' => 'ru'

config/app.php

$date = Carbon::now(); $dayName = $date->getTranslatedDayName(); // 'Monday', but without the package it would be 'Понедельник'

DaData v2

DaData V2

Dear community, a planing make a new version for DaData Laravel package. It's main issue for working with DaData integration. Grow speed and fixing all bugs.

• Type cast for all models
• New DaData API interface
• New tests
• Bugfixing

Ошибка в методе DaDataCompany::prompt

movemoveapp/laravel-dadata: 1.0.10
Передаю параметр string $locations = '66' в метод, а в сформированном запросе на выходе из метода
{
"query": "Федерация акро",
"count": 1,
"status": [ "ACTIVE" ],
"type": "LEGAL",
"locations": [ { "kladr_id": "6" } ],
"locations_boost": []
}
Одна шестерка потерялась, тоже самое происходит со всеми двузначными цифрами. Нужно изменить код немного в этом методе.

    if (! is_null($locations)) {
        $locations = explode(',', $locations);
        for ($i = 0; $i < count($locations); $i++) {
            array_push($locations_array, ['kladr_id' => trim($locations[$i])]);
        }
    }

Company search always with legal entity or individual filter

At the moment, only a crutch can be used to search for companies, both individuals and legal entities, for this you need to pass a value that does not exist in CompanyType, since null cannot be passed. This is wrong, you need to add support so that when searching for a company, you should not be limited to searching only for legal entities or individuals. Just add support for int|null in the DaDataCompany::prompt(type: null) argument

laravel-dadata/src/DaDataCompany.php

Line 61 in 090e5d9

int $type = CompanyType::INDIVIDUAL,

Поддержка PHP 8.0

Привет, нужно добавить поддержку PHP 8.0 )

Версия laravel

Добрый день.

В laravel 5.8 пакет illuminate/support такой же версии 5.8

для composer require movemoveapp/laravel-dadata - requires illuminate/support ^8.8.0

Получается ваш пакет встанет только на laravel 8. Просто в readme написано что делать для версий ниже 5.5 , а по факту оно туда и не установится.

DaDataAddress::prompt() не поддерживает restrict_value

если в запрос добавить флаг "restrict_value": true, адреса не будут содержать часть адреса родителя, если он задан как фильтр. Библиотека это не поддерживает

пример из конфлюенса https://confluence.hflabs.ru/pages/viewpage.action?pageId=1023737934

Using $this when not in object context Laravel Dadata

Добрый день установил плагин с игнор версией, а также пробовал устанавливать версию от ребят из сушимаркета (https://github.com/sushi-market/laravel-dadata) но всё равно выбивает эту ошибку при запросе:

exception: "Error"
file: "/home/cotit/dev3/vendor/movemoveapp/laravel-dadata/src/DaDataAddress.php"
line: 59
message: "Using $this when not in object context"

В чём проблема?

Ошибка в методе id class DaDataCompany

Добрый день. В методе параметр $type ожидается как CompanyType, а в сам запрос отправляется как CompanyStatus.

public function id(
        string $id,
        int $count = 10,
        string $kpp = null,
        int $branch_type = BranchType::MAIN,
        int $type = CompanyType::LEGAL
    ): array {
        dd(CompanyStatus::$map[$type]);
        return $this->suggestApi()->post('rs/findById/party', [
            'query'             => $id,
            'count'             => $count,
            'kpp'               => $kpp,
            'branch_type'       => BranchType::$map[$branch_type] ?? BranchType::$map[BranchType::MAIN],
            'type'              => CompanyStatus::$map[$type] ?? null,
        ]);
    }

Условия поиска по ИНН или ОГРН

В документации указано что поиск будет произведен по всем филиалам, то есть по умолчанию не указывается branch_type
https://dadata.ru/api/find-party/

У вас в коде
https://github.com/movemoveapp/laravel-dadata/blob/master/src/DaDataCompany.php#L39
По умолчанию MAIN (можно изменить на BRANCH) но нельзя указать null, чтобы не передавалось и был произведен поиск по всем филиалам

Возможно я не до конца разобрался в документации, но если это так исправьте пожалуйста

INDIVIDUAL check error

Hi!

In Providers/DaDataCompany.php on line 40 there is error (which you can catch if you try to check INDIVIDUAL by INN number).
On property 'type' - wrong ENUM in value. Instead of "CompanyType::$map[$type] ?? null" there is "CompanyStatus::$map[$type] ?? null" (status instead of type).

I've fixed it manually, but it would be better to fix this in release.

Нет методов для получения баланса и остатка запросов

В пакете отсутствуют методы для получения баланса (profile/balance) и остатка запросов (stat/daily)

Token error if config:cache

hi there

after:
artisan config:cache

or:
artisan optimize

got:
exception: "TypeError" file: "/var/www/html/vendor/movemoveapp/laravel-dadata/src/Providers/CleanerDaDataProvider.php" line: 52 message: "MoveMoveIo\\DaData\\Providers\\CleanerDaDataProvider::__construct(): Argument #1 ($token) must be of type string, null given, called in /var/www/html/vendor/movemoveapp/laravel-dadata/src/DaDataService.php on line 88"