diff --git a/application/Espo/Core/Select/SelectManager.php b/application/Espo/Core/Select/SelectManager.php index a8c88caa31..6967bf98fa 100644 --- a/application/Espo/Core/Select/SelectManager.php +++ b/application/Espo/Core/Select/SelectManager.php @@ -323,7 +323,7 @@ class SelectManager /** * Convert 'where' parameters from the frontend format to the format needed by ORM. * - * @return Where clause for ORM. + * @return array Where clause for ORM. */ public function convertWhere(array $where, bool $ignoreAdditionaFilterTypes = false, array &$result = []) : array { diff --git a/application/Espo/Tools/EntityManager/EntityManager.php b/application/Espo/Tools/EntityManager/EntityManager.php index 2987d665a9..d25f8d2a70 100644 --- a/application/Espo/Tools/EntityManager/EntityManager.php +++ b/application/Espo/Tools/EntityManager/EntityManager.php @@ -1225,6 +1225,7 @@ class EntityManager * foreignLinkEntityTypeList?: string[], * } $params * @throws BadRequest + * @throws Error */ public function updateLink(array $params): void { @@ -1542,11 +1543,11 @@ class EntityManager /** * @param array $data + * @throws Error */ public function setFormulaData(string $scope, array $data): void { $this->metadata->set('formula', $scope, $data); - $this->metadata->save(); $this->dataManager->clearCache(); diff --git a/application/Espo/Tools/Export/Processors/Xlsx.php b/application/Espo/Tools/Export/Processors/Xlsx.php index 80f78bb034..052f5fb891 100644 --- a/application/Espo/Tools/Export/Processors/Xlsx.php +++ b/application/Espo/Tools/Export/Processors/Xlsx.php @@ -125,6 +125,10 @@ class Xlsx implements Processor $this->addressFormatterFactory = $addressFormatterFactory; } + /** + * @throws \PhpOffice\PhpSpreadsheet\Exception + * @throws \PhpOffice\PhpSpreadsheet\Writer\Exception + */ public function process(Params $params, Data $data): StreamInterface { $entityType = $params->getEntityType(); @@ -368,6 +372,7 @@ class Xlsx implements Processor * @param string[] $fieldList * @param string[] $azRange * @param array $typesCache + * @throws \PhpOffice\PhpSpreadsheet\Exception */ private function processRow( string $entityType, diff --git a/application/Espo/Tools/FieldManager/FieldManager.php b/application/Espo/Tools/FieldManager/FieldManager.php index a93bfccc7f..56da2c7c37 100644 --- a/application/Espo/Tools/FieldManager/FieldManager.php +++ b/application/Espo/Tools/FieldManager/FieldManager.php @@ -114,6 +114,7 @@ class FieldManager /** * @return array + * @throws Error */ public function read(string $scope, string $name): array { diff --git a/application/Espo/Tools/Import/Import.php b/application/Espo/Tools/Import/Import.php index a8aecf7a33..b754286c3f 100644 --- a/application/Espo/Tools/Import/Import.php +++ b/application/Espo/Tools/Import/Import.php @@ -188,6 +188,9 @@ class Import return $this; } + /** + * @throws Error + */ private function validate(): void { if (!$this->entityType) { @@ -201,6 +204,7 @@ class Import /** * Run import. + * @throws Error */ public function run(): Result { diff --git a/application/Espo/Tools/Import/Service.php b/application/Espo/Tools/Import/Service.php index 1b1b062b11..276d81b3bf 100644 --- a/application/Espo/Tools/Import/Service.php +++ b/application/Espo/Tools/Import/Service.php @@ -72,6 +72,7 @@ class Service * @param string[] $attributeList * @param string $attachmentId * @throws Forbidden + * @throws Error */ public function import( string $entityType, @@ -107,6 +108,10 @@ class Service return $result; } + /** + * @throws Forbidden + * @throws Error + */ public function importContentsWithParamsId(string $contents, string $importParamsId): Result { if (!$contents) { diff --git a/application/Espo/Tools/LayoutManager/LayoutManager.php b/application/Espo/Tools/LayoutManager/LayoutManager.php index 4dc7cb1bd3..21129d77a5 100644 --- a/application/Espo/Tools/LayoutManager/LayoutManager.php +++ b/application/Espo/Tools/LayoutManager/LayoutManager.php @@ -68,6 +68,7 @@ class LayoutManager * Set layout data. * * @param mixed $data + * @throws Error */ public function set($data, string $scope, string $name): void { diff --git a/application/Espo/Tools/MassUpdate/MassUpdate.php b/application/Espo/Tools/MassUpdate/MassUpdate.php index 3b771680d2..e3cfd8660d 100644 --- a/application/Espo/Tools/MassUpdate/MassUpdate.php +++ b/application/Espo/Tools/MassUpdate/MassUpdate.php @@ -60,6 +60,7 @@ class MassUpdate /** * @param ?User $user Under what user to perform mass-update. If not specified, the system user will be used. * Access control is applied for the user. + * @throws \Espo\Core\Exceptions\NotFound */ public function process(Params $params, Data $data, ?User $user = null): Result { diff --git a/application/Espo/Tools/Pdf/PrinterController.php b/application/Espo/Tools/Pdf/PrinterController.php index 1e90bd188d..8a7e6b83de 100644 --- a/application/Espo/Tools/Pdf/PrinterController.php +++ b/application/Espo/Tools/Pdf/PrinterController.php @@ -73,6 +73,7 @@ class PrinterController /** * @param Collection $collection + * @throws Error */ public function printCollection(Collection $collection, ?Params $params, ?IdDataMap $IdDataMap = null): Contents { @@ -82,6 +83,9 @@ class PrinterController return $this->createCollectionPrinter()->print($this->template, $collection, $params, $IdDataMap); } + /** + * @throws Error + */ private function createEntityPrinter(): EntityPrinter { /** @var ?class-string */ @@ -95,6 +99,9 @@ class PrinterController return $this->injectableFactory->create($className); } + /** + * @throws Error + */ private function createCollectionPrinter(): CollectionPrinter { /** @var ?class-string */ diff --git a/application/Espo/Tools/Pdf/Tcpdf/Tcpdf.php b/application/Espo/Tools/Pdf/Tcpdf/Tcpdf.php index 77bc24b426..a89170f04f 100644 --- a/application/Espo/Tools/Pdf/Tcpdf/Tcpdf.php +++ b/application/Espo/Tools/Pdf/Tcpdf/Tcpdf.php @@ -184,6 +184,7 @@ class Tcpdf extends TcpdfOriginal * @param string $name * @param string $dest * @return string + * @throws \Exception */ public function Output($name = 'doc.pdf', $dest = 'I') {