From f8464cf72c93dc18b4ba4ac29e6d28b74b73e7fb Mon Sep 17 00:00:00 2001 From: GregM Date: Wed, 15 Jan 2025 12:12:23 +0100 Subject: [PATCH] add type param and return function --- .../box_accountancy_last_manual_entries.php | 2 +- .../box_accountancy_suspense_account.php | 2 +- htdocs/core/boxes/box_actions.php | 2 +- htdocs/core/boxes/box_actions_future.php | 2 +- htdocs/core/boxes/box_activity.php | 2 +- htdocs/core/boxes/box_birthdays.php | 2 +- htdocs/core/boxes/box_birthdays_members.php | 2 +- htdocs/core/boxes/box_boms.php | 2 +- htdocs/core/boxes/box_bookmarks.php | 2 +- htdocs/core/boxes/box_clients.php | 2 +- htdocs/core/boxes/box_commandes.php | 2 +- htdocs/core/boxes/box_comptes.php | 2 +- htdocs/core/boxes/box_contacts.php | 2 +- htdocs/core/boxes/box_contracts.php | 2 +- ...box_customers_outstanding_bill_reached.php | 2 +- .../core/boxes/box_dolibarr_state_board.php | 2 +- htdocs/core/boxes/box_external_rss.php | 2 +- htdocs/core/boxes/box_factures.php | 2 +- htdocs/core/boxes/box_factures_fourn.php | 2 +- htdocs/core/boxes/box_factures_fourn_imp.php | 2 +- htdocs/core/boxes/box_factures_imp.php | 2 +- htdocs/core/boxes/box_fediverse.php | 2 +- htdocs/core/boxes/box_ficheinter.php | 2 +- htdocs/core/boxes/box_fournisseurs.php | 2 +- .../core/boxes/box_funnel_of_prospection.php | 2 +- htdocs/core/boxes/box_goodcustomers.php | 2 +- .../boxes/box_graph_invoices_permonth.php | 2 +- .../core/boxes/box_graph_invoices_peryear.php | 2 +- .../box_graph_invoices_supplier_permonth.php | 2 +- .../boxes/box_graph_nb_ticket_last_x_days.php | 2 +- .../core/boxes/box_graph_nb_tickets_type.php | 2 +- .../boxes/box_graph_new_vs_close_ticket.php | 2 +- .../core/boxes/box_graph_orders_permonth.php | 2 +- .../box_graph_orders_supplier_permonth.php | 2 +- .../boxes/box_graph_product_distribution.php | 2 +- .../boxes/box_graph_propales_permonth.php | 2 +- .../boxes/box_graph_ticket_by_severity.php | 2 +- .../core/boxes/box_last_knowledgerecord.php | 2 +- .../box_last_modified_knowledgerecord.php | 2 +- .../core/boxes/box_last_modified_ticket.php | 2 +- htdocs/core/boxes/box_last_ticket.php | 2 +- htdocs/core/boxes/box_lastlogin.php | 2 +- htdocs/core/boxes/box_members_by_tags.php | 2 +- htdocs/core/boxes/box_members_by_type.php | 2 +- .../core/boxes/box_members_last_modified.php | 2 +- .../boxes/box_members_last_subscriptions.php | 2 +- .../box_members_subscriptions_by_year.php | 2 +- htdocs/core/boxes/box_mos.php | 2 +- htdocs/core/boxes/box_produits.php | 2 +- .../core/boxes/box_produits_alerte_stock.php | 2 +- htdocs/core/boxes/box_project.php | 2 +- .../core/boxes/box_project_opportunities.php | 2 +- htdocs/core/boxes/box_propales.php | 2 +- htdocs/core/boxes/box_prospect.php | 2 +- htdocs/core/boxes/box_scheduled_jobs.php | 2 +- htdocs/core/boxes/box_services_contracts.php | 2 +- htdocs/core/boxes/box_services_expired.php | 2 +- htdocs/core/boxes/box_shipments.php | 2 +- htdocs/core/boxes/box_supplier_orders.php | 2 +- ...box_supplier_orders_awaiting_reception.php | 2 +- htdocs/core/boxes/box_task.php | 2 +- htdocs/core/boxes/box_validated_projects.php | 2 +- htdocs/core/boxes/modules_boxes.php | 4 +- .../class/commonnumrefgenerator.class.php | 12 ++--- .../doc/doc_generic_asset_odt.modules.php | 2 +- .../core/modules/asset/mod_asset_advanced.php | 2 +- .../core/modules/asset/mod_asset_standard.php | 4 +- .../barcode/doc/phpbarcode.modules.php | 4 +- .../barcode/doc/tcpdfbarcode.modules.php | 4 +- .../barcode/mod_barcode_product_standard.php | 2 +- .../mod_barcode_thirdparty_standard.php | 2 +- .../modules/barcode/modules_barcode.class.php | 2 +- .../bom/doc/doc_generic_bom_odt.modules.php | 2 +- htdocs/core/modules/bom/mod_bom_advanced.php | 2 +- htdocs/core/modules/bom/mod_bom_standard.php | 4 +- .../modules/cheque/mod_chequereceipt_mint.php | 4 +- .../cheque/mod_chequereceipt_thyme.php | 2 +- .../doc/doc_generic_order_odt.modules.php | 2 +- .../modules/commande/mod_commande_marbre.php | 4 +- .../modules/commande/mod_commande_saphir.php | 2 +- .../doc/doc_generic_contract_odt.modules.php | 2 +- .../modules/contract/mod_contract_magre.php | 2 +- .../modules/contract/mod_contract_olive.php | 2 +- .../modules/contract/mod_contract_serpis.php | 4 +- .../modules/delivery/mod_delivery_jade.php | 4 +- .../modules/delivery/mod_delivery_saphir.php | 2 +- .../doc/doc_generic_shipment_odt.modules.php | 2 +- .../expedition/mod_expedition_ribera.php | 2 +- .../expedition/mod_expedition_safor.php | 4 +- .../expensereport/mod_expensereport_jade.php | 4 +- .../expensereport/mod_expensereport_sand.php | 2 +- .../doc/doc_generic_invoice_odt.modules.php | 2 +- .../core/modules/facture/mod_facture_mars.php | 4 +- .../modules/facture/mod_facture_mercure.php | 2 +- .../modules/facture/mod_facture_terre.php | 4 +- htdocs/core/modules/fichinter/mod_arctic.php | 2 +- htdocs/core/modules/fichinter/mod_pacific.php | 4 +- .../holiday/mod_holiday_immaculate.php | 2 +- .../modules/holiday/mod_holiday_madonna.php | 4 +- .../modules/hrm/mod_evaluation_advanced.php | 2 +- .../modules/hrm/mod_evaluation_standard.php | 4 +- .../modules/mailings/modules_mailings.php | 22 ++++----- .../doc/doc_generic_member_odt.class.php | 2 +- .../modules/member/mod_member_advanced.php | 4 +- .../core/modules/member/mod_member_simple.php | 4 +- .../mrp/doc/doc_generic_mo_odt.modules.php | 2 +- htdocs/core/modules/mrp/mod_mo_advanced.php | 2 +- htdocs/core/modules/mrp/mod_mo_standard.php | 4 +- .../core/modules/payment/mod_payment_ant.php | 2 +- .../modules/payment/mod_payment_cicada.php | 4 +- .../doc/doc_generic_product_odt.modules.php | 2 +- .../product/mod_codeproduct_elephant.php | 2 +- .../product/mod_codeproduct_leopard.php | 2 +- .../product_batch/mod_lot_advanced.php | 2 +- .../modules/product_batch/mod_lot_free.php | 2 +- .../product_batch/mod_lot_standard.php | 4 +- .../modules/product_batch/mod_sn_advanced.php | 2 +- .../modules/product_batch/mod_sn_free.php | 2 +- .../modules/product_batch/mod_sn_standard.php | 4 +- .../doc/doc_generic_project_odt.modules.php | 2 +- .../modules/project/mod_project_simple.php | 4 +- .../modules/project/mod_project_universal.php | 2 +- .../task/doc/doc_generic_task_odt.modules.php | 2 +- .../modules/project/task/mod_task_simple.php | 4 +- .../project/task/mod_task_universal.php | 2 +- .../doc/doc_generic_proposal_odt.modules.php | 2 +- .../modules/propale/mod_propale_marbre.php | 4 +- .../modules/propale/mod_propale_saphir.php | 2 +- .../doc/doc_generic_reception_odt.modules.php | 2 +- .../modules/reception/mod_reception_beryl.php | 4 +- .../reception/mod_reception_moonstone.php | 2 +- .../societe/doc/doc_generic_odt.modules.php | 2 +- .../societe/mod_codeclient_elephant.php | 2 +- .../societe/mod_codeclient_leopard.php | 2 +- .../modules/societe/mod_codeclient_monkey.php | 2 +- .../societe/mod_codecompta_aquarium.php | 2 +- .../societe/mod_codecompta_digitaria.php | 2 +- .../societe/mod_codecompta_panicum.php | 2 +- .../doc/doc_generic_stock_odt.modules.php | 2 +- .../mod_stocktransfer_advanced.php | 2 +- .../mod_stocktransfer_standard.php | 4 +- ...c_generic_supplier_invoice_odt.modules.php | 2 +- .../mod_facture_fournisseur_cactus.php | 4 +- .../mod_facture_fournisseur_tulip.php | 2 +- ...doc_generic_supplier_order_odt.modules.php | 2 +- .../mod_commande_fournisseur_muguet.php | 4 +- .../mod_commande_fournisseur_orchidee.php | 2 +- .../mod_supplier_payment_brodator.php | 2 +- .../mod_supplier_payment_bronan.php | 4 +- ..._generic_supplier_proposal_odt.modules.php | 2 +- .../mod_supplier_proposal_marbre.php | 4 +- .../mod_supplier_proposal_saphir.php | 2 +- .../takepos/mod_takepos_ref_simple.php | 4 +- .../takepos/mod_takepos_ref_universal.php | 2 +- .../doc/doc_generic_ticket_odt.modules.php | 2 +- .../core/modules/ticket/mod_ticket_simple.php | 4 +- .../modules/ticket/mod_ticket_universal.php | 2 +- .../user/doc/doc_generic_user_odt.modules.php | 2 +- .../doc/doc_generic_usergroup_odt.modules.php | 2 +- .../workstation/mod_workstation_advanced.php | 2 +- .../workstation/mod_workstation_standard.php | 4 +- .../core/triggers/dolibarrtriggers.class.php | 12 ++--- .../interface_20_all_Logevents.class.php | 2 +- ...e_20_modWorkflow_WorkflowManager.class.php | 2 +- ...terface_50_modAgenda_ActionsAuto.class.php | 2 +- ..._modBlockedlog_ActionsBlockedLog.class.php | 2 +- ...ntOrganization_EventOrganization.class.php | 2 +- ...interface_50_modLdap_Ldapsynchro.class.php | 2 +- ...odMailmanspip_Mailmanspipsynchro.class.php | 2 +- ..._50_modNotification_Notification.class.php | 2 +- ...terface_50_modTicket_TicketEmail.class.php | 2 +- .../interface_80_modStripe_Stripe.class.php | 2 +- ...rface_90_modSociete_ContactRoles.class.php | 2 +- ...ce_95_modWebhook_WebhookTriggers.class.php | 2 +- ...face_95_modZapier_ZapierTriggers.class.php | 2 +- .../mod_knowledgerecord_standard.php | 2 +- .../modules_knowledgerecord.php | 2 +- .../template/class/actions_mymodule.class.php | 20 ++++---- .../template/class/api_mymodule.class.php | 12 ++--- .../template/class/myobject.class.php | 48 +++++++++---------- .../template/core/boxes/mymodulewidget1.php | 6 +-- .../mailing_mymodule_selector1.modules.php | 12 ++--- .../mymodule/mod_myobject_advanced.php | 6 +-- .../mymodule/mod_myobject_standard.php | 8 ++-- .../modules/mymodule/modules_myobject.php | 8 ++-- ..._99_modMyModule_MyModuleTriggers.class.php | 4 +- .../template/lib/mymodule.lib.php | 2 +- .../template/lib/mymodule_myobject.lib.php | 2 +- .../partnership/mod_partnership_advanced.php | 2 +- .../partnership/mod_partnership_standard.php | 4 +- .../mod_recruitmentcandidature_advanced.php | 2 +- .../mod_recruitmentcandidature_standard.php | 4 +- .../mod_recruitmentjobposition_advanced.php | 2 +- .../mod_recruitmentjobposition_standard.php | 4 +- 194 files changed, 304 insertions(+), 304 deletions(-) diff --git a/htdocs/core/boxes/box_accountancy_last_manual_entries.php b/htdocs/core/boxes/box_accountancy_last_manual_entries.php index cad285014c233..d17eaf3553fba 100644 --- a/htdocs/core/boxes/box_accountancy_last_manual_entries.php +++ b/htdocs/core/boxes/box_accountancy_last_manual_entries.php @@ -59,7 +59,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_accountancy_suspense_account.php b/htdocs/core/boxes/box_accountancy_suspense_account.php index 1bf03ec68d64d..dc3868e581e38 100644 --- a/htdocs/core/boxes/box_accountancy_suspense_account.php +++ b/htdocs/core/boxes/box_accountancy_suspense_account.php @@ -59,7 +59,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 1) + public function loadBox(int $max = 1): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_actions.php b/htdocs/core/boxes/box_actions.php index 17151fe3a4af0..ebc821263a359 100644 --- a/htdocs/core/boxes/box_actions.php +++ b/htdocs/core/boxes/box_actions.php @@ -66,7 +66,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; diff --git a/htdocs/core/boxes/box_actions_future.php b/htdocs/core/boxes/box_actions_future.php index 5d4e8dcd2e85a..8e417a390dd66 100644 --- a/htdocs/core/boxes/box_actions_future.php +++ b/htdocs/core/boxes/box_actions_future.php @@ -63,7 +63,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_activity.php b/htdocs/core/boxes/box_activity.php index 9ab9544671e0d..e7e88e6f5f3e3 100644 --- a/htdocs/core/boxes/box_activity.php +++ b/htdocs/core/boxes/box_activity.php @@ -66,7 +66,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_birthdays.php b/htdocs/core/boxes/box_birthdays.php index 30cc8ea9491ef..a06632c98156c 100644 --- a/htdocs/core/boxes/box_birthdays.php +++ b/htdocs/core/boxes/box_birthdays.php @@ -61,7 +61,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 20) + public function loadBox(int $max = 20): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_birthdays_members.php b/htdocs/core/boxes/box_birthdays_members.php index edf35d08df134..43e32d361068a 100644 --- a/htdocs/core/boxes/box_birthdays_members.php +++ b/htdocs/core/boxes/box_birthdays_members.php @@ -61,7 +61,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 20) + public function loadBox(int $max = 20): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_boms.php b/htdocs/core/boxes/box_boms.php index 75d929e0b1ae9..db44520244364 100644 --- a/htdocs/core/boxes/box_boms.php +++ b/htdocs/core/boxes/box_boms.php @@ -61,7 +61,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_bookmarks.php b/htdocs/core/boxes/box_bookmarks.php index dea4675db3f4b..1d9ab59bc3d07 100644 --- a/htdocs/core/boxes/box_bookmarks.php +++ b/htdocs/core/boxes/box_bookmarks.php @@ -57,7 +57,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_clients.php b/htdocs/core/boxes/box_clients.php index 9825232ba948c..3e7972c18233b 100644 --- a/htdocs/core/boxes/box_clients.php +++ b/htdocs/core/boxes/box_clients.php @@ -69,7 +69,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $hookmanager; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_commandes.php b/htdocs/core/boxes/box_commandes.php index e4ea864271a8a..71fd4e6942633 100644 --- a/htdocs/core/boxes/box_commandes.php +++ b/htdocs/core/boxes/box_commandes.php @@ -63,7 +63,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5):void { global $user, $langs, $conf; $langs->load('orders'); diff --git a/htdocs/core/boxes/box_comptes.php b/htdocs/core/boxes/box_comptes.php index f67bf006148f9..1e99be81060a7 100644 --- a/htdocs/core/boxes/box_comptes.php +++ b/htdocs/core/boxes/box_comptes.php @@ -70,7 +70,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_contacts.php b/htdocs/core/boxes/box_contacts.php index 6db28aca4f9c6..35e3986f95366 100644 --- a/htdocs/core/boxes/box_contacts.php +++ b/htdocs/core/boxes/box_contacts.php @@ -65,7 +65,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf, $hookmanager; diff --git a/htdocs/core/boxes/box_contracts.php b/htdocs/core/boxes/box_contracts.php index e7700692e3eb8..11080c1727702 100644 --- a/htdocs/core/boxes/box_contracts.php +++ b/htdocs/core/boxes/box_contracts.php @@ -60,7 +60,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_customers_outstanding_bill_reached.php b/htdocs/core/boxes/box_customers_outstanding_bill_reached.php index 8dcd6a5ecc7e9..7769c231bb800 100644 --- a/htdocs/core/boxes/box_customers_outstanding_bill_reached.php +++ b/htdocs/core/boxes/box_customers_outstanding_bill_reached.php @@ -66,7 +66,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_dolibarr_state_board.php b/htdocs/core/boxes/box_dolibarr_state_board.php index 13d563c60dc0d..300f2d24e3ac5 100644 --- a/htdocs/core/boxes/box_dolibarr_state_board.php +++ b/htdocs/core/boxes/box_dolibarr_state_board.php @@ -57,7 +57,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_external_rss.php b/htdocs/core/boxes/box_external_rss.php index 688396580aeba..977f3a2ccaa46 100644 --- a/htdocs/core/boxes/box_external_rss.php +++ b/htdocs/core/boxes/box_external_rss.php @@ -65,7 +65,7 @@ public function __construct($db, $param) * @param int $cachedelay Delay we accept for cache file * @return void */ - public function loadBox($max = 5, $cachedelay = 3600) + public function loadBox(int $max = 5,int $cachedelay = 3600): void { global $user, $langs, $conf; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_factures.php b/htdocs/core/boxes/box_factures.php index ee333e1eec53f..400a92204885f 100644 --- a/htdocs/core/boxes/box_factures.php +++ b/htdocs/core/boxes/box_factures.php @@ -59,7 +59,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_factures_fourn.php b/htdocs/core/boxes/box_factures_fourn.php index 511a041a0359d..f44012cf860c6 100644 --- a/htdocs/core/boxes/box_factures_fourn.php +++ b/htdocs/core/boxes/box_factures_fourn.php @@ -60,7 +60,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_factures_fourn_imp.php b/htdocs/core/boxes/box_factures_fourn_imp.php index 21ef52a56fc83..0e12b856bfac8 100644 --- a/htdocs/core/boxes/box_factures_fourn_imp.php +++ b/htdocs/core/boxes/box_factures_fourn_imp.php @@ -57,7 +57,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_factures_imp.php b/htdocs/core/boxes/box_factures_imp.php index fb7e8f914ea13..ada4751eaf638 100644 --- a/htdocs/core/boxes/box_factures_imp.php +++ b/htdocs/core/boxes/box_factures_imp.php @@ -63,7 +63,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_fediverse.php b/htdocs/core/boxes/box_fediverse.php index b4cd84fc64260..025521358c0df 100644 --- a/htdocs/core/boxes/box_fediverse.php +++ b/htdocs/core/boxes/box_fediverse.php @@ -61,7 +61,7 @@ public function __construct($db, $param) * @param int $cachedelay Delay we accept for cache file * @return void */ - public function loadBox($max = 5, $cachedelay = 3600) + public function loadBox(int $max = 5,int $cachedelay = 3600): void { global $langs; diff --git a/htdocs/core/boxes/box_ficheinter.php b/htdocs/core/boxes/box_ficheinter.php index ff12319eebe4e..042dbbff0a119 100644 --- a/htdocs/core/boxes/box_ficheinter.php +++ b/htdocs/core/boxes/box_ficheinter.php @@ -60,7 +60,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 10) + public function loadBox(int $max = 10): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_fournisseurs.php b/htdocs/core/boxes/box_fournisseurs.php index 6cd7b8c3bab7d..ce7d8c5d26011 100644 --- a/htdocs/core/boxes/box_fournisseurs.php +++ b/htdocs/core/boxes/box_fournisseurs.php @@ -61,7 +61,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $hookmanager; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_funnel_of_prospection.php b/htdocs/core/boxes/box_funnel_of_prospection.php index d6cff6b37c473..d6ac518693c6c 100644 --- a/htdocs/core/boxes/box_funnel_of_prospection.php +++ b/htdocs/core/boxes/box_funnel_of_prospection.php @@ -67,7 +67,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf; diff --git a/htdocs/core/boxes/box_goodcustomers.php b/htdocs/core/boxes/box_goodcustomers.php index 12d12b7e855ae..13e9b2344e1b5 100644 --- a/htdocs/core/boxes/box_goodcustomers.php +++ b/htdocs/core/boxes/box_goodcustomers.php @@ -70,7 +70,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_graph_invoices_permonth.php b/htdocs/core/boxes/box_graph_invoices_permonth.php index af0dcbac27770..c5b236aef84f1 100644 --- a/htdocs/core/boxes/box_graph_invoices_permonth.php +++ b/htdocs/core/boxes/box_graph_invoices_permonth.php @@ -59,7 +59,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_invoices_peryear.php b/htdocs/core/boxes/box_graph_invoices_peryear.php index fccfa5b7530f9..1cb988e7d15ab 100644 --- a/htdocs/core/boxes/box_graph_invoices_peryear.php +++ b/htdocs/core/boxes/box_graph_invoices_peryear.php @@ -56,7 +56,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_invoices_supplier_permonth.php b/htdocs/core/boxes/box_graph_invoices_supplier_permonth.php index f3d556b093306..d072c2e7782b7 100644 --- a/htdocs/core/boxes/box_graph_invoices_supplier_permonth.php +++ b/htdocs/core/boxes/box_graph_invoices_supplier_permonth.php @@ -58,7 +58,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_nb_ticket_last_x_days.php b/htdocs/core/boxes/box_graph_nb_ticket_last_x_days.php index 55ba045135145..dd1edace81864 100644 --- a/htdocs/core/boxes/box_graph_nb_ticket_last_x_days.php +++ b/htdocs/core/boxes/box_graph_nb_ticket_last_x_days.php @@ -61,7 +61,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; $dataseries = array(); diff --git a/htdocs/core/boxes/box_graph_nb_tickets_type.php b/htdocs/core/boxes/box_graph_nb_tickets_type.php index 4e6d69a298a1f..2a03a4edf9fee 100644 --- a/htdocs/core/boxes/box_graph_nb_tickets_type.php +++ b/htdocs/core/boxes/box_graph_nb_tickets_type.php @@ -61,7 +61,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; global $theme_datacolor, $badgeStatus8; diff --git a/htdocs/core/boxes/box_graph_new_vs_close_ticket.php b/htdocs/core/boxes/box_graph_new_vs_close_ticket.php index 1c29d20bcbf5c..c7426ecf43c23 100644 --- a/htdocs/core/boxes/box_graph_new_vs_close_ticket.php +++ b/htdocs/core/boxes/box_graph_new_vs_close_ticket.php @@ -61,7 +61,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_orders_permonth.php b/htdocs/core/boxes/box_graph_orders_permonth.php index 3a3cb66df2d86..849c6d479e587 100644 --- a/htdocs/core/boxes/box_graph_orders_permonth.php +++ b/htdocs/core/boxes/box_graph_orders_permonth.php @@ -58,7 +58,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_orders_supplier_permonth.php b/htdocs/core/boxes/box_graph_orders_supplier_permonth.php index 0f0e9f2df6f50..cbb582d0e9931 100644 --- a/htdocs/core/boxes/box_graph_orders_supplier_permonth.php +++ b/htdocs/core/boxes/box_graph_orders_supplier_permonth.php @@ -58,7 +58,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_product_distribution.php b/htdocs/core/boxes/box_graph_product_distribution.php index e405727d7ffff..4c07a46e99ad8 100644 --- a/htdocs/core/boxes/box_graph_product_distribution.php +++ b/htdocs/core/boxes/box_graph_product_distribution.php @@ -62,7 +62,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_propales_permonth.php b/htdocs/core/boxes/box_graph_propales_permonth.php index 7e2fb8d27d184..e6023d968354e 100644 --- a/htdocs/core/boxes/box_graph_propales_permonth.php +++ b/htdocs/core/boxes/box_graph_propales_permonth.php @@ -58,7 +58,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_graph_ticket_by_severity.php b/htdocs/core/boxes/box_graph_ticket_by_severity.php index 2eae2666c85be..c8cc0db077283 100644 --- a/htdocs/core/boxes/box_graph_ticket_by_severity.php +++ b/htdocs/core/boxes/box_graph_ticket_by_severity.php @@ -61,7 +61,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_last_knowledgerecord.php b/htdocs/core/boxes/box_last_knowledgerecord.php index 325c8cb868350..547efd92fc51a 100644 --- a/htdocs/core/boxes/box_last_knowledgerecord.php +++ b/htdocs/core/boxes/box_last_knowledgerecord.php @@ -71,7 +71,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; diff --git a/htdocs/core/boxes/box_last_modified_knowledgerecord.php b/htdocs/core/boxes/box_last_modified_knowledgerecord.php index c41d141d5e8d8..4ace1c95fd142 100644 --- a/htdocs/core/boxes/box_last_modified_knowledgerecord.php +++ b/htdocs/core/boxes/box_last_modified_knowledgerecord.php @@ -71,7 +71,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; diff --git a/htdocs/core/boxes/box_last_modified_ticket.php b/htdocs/core/boxes/box_last_modified_ticket.php index 7016d62449c05..e58b46f1688e7 100644 --- a/htdocs/core/boxes/box_last_modified_ticket.php +++ b/htdocs/core/boxes/box_last_modified_ticket.php @@ -59,7 +59,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; diff --git a/htdocs/core/boxes/box_last_ticket.php b/htdocs/core/boxes/box_last_ticket.php index f251cd944bfad..ed0a2b7334608 100644 --- a/htdocs/core/boxes/box_last_ticket.php +++ b/htdocs/core/boxes/box_last_ticket.php @@ -59,7 +59,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_lastlogin.php b/htdocs/core/boxes/box_lastlogin.php index 6ec1d99c03216..5f9e72f6a1f3d 100644 --- a/htdocs/core/boxes/box_lastlogin.php +++ b/htdocs/core/boxes/box_lastlogin.php @@ -57,7 +57,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_members_by_tags.php b/htdocs/core/boxes/box_members_by_tags.php index 07a9e9962ad8c..54bfc7f541b4e 100644 --- a/htdocs/core/boxes/box_members_by_tags.php +++ b/htdocs/core/boxes/box_members_by_tags.php @@ -68,7 +68,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_members_by_type.php b/htdocs/core/boxes/box_members_by_type.php index ebe15471a4b67..b3b6d3a7960b9 100644 --- a/htdocs/core/boxes/box_members_by_type.php +++ b/htdocs/core/boxes/box_members_by_type.php @@ -68,7 +68,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; $langs->loadLangs(array("boxes", "members")); diff --git a/htdocs/core/boxes/box_members_last_modified.php b/htdocs/core/boxes/box_members_last_modified.php index c4a3fc27e7d9a..3dedd403d64cc 100644 --- a/htdocs/core/boxes/box_members_last_modified.php +++ b/htdocs/core/boxes/box_members_last_modified.php @@ -67,7 +67,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_members_last_subscriptions.php b/htdocs/core/boxes/box_members_last_subscriptions.php index f6323dd7af929..77e3ddcedf877 100644 --- a/htdocs/core/boxes/box_members_last_subscriptions.php +++ b/htdocs/core/boxes/box_members_last_subscriptions.php @@ -70,7 +70,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_members_subscriptions_by_year.php b/htdocs/core/boxes/box_members_subscriptions_by_year.php index 755e58a011854..832c06790e616 100644 --- a/htdocs/core/boxes/box_members_subscriptions_by_year.php +++ b/htdocs/core/boxes/box_members_subscriptions_by_year.php @@ -67,7 +67,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; $langs->loadLangs(array("boxes", "members")); diff --git a/htdocs/core/boxes/box_mos.php b/htdocs/core/boxes/box_mos.php index be80c7d4536e1..ff27ee5a2b850 100644 --- a/htdocs/core/boxes/box_mos.php +++ b/htdocs/core/boxes/box_mos.php @@ -62,7 +62,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_produits.php b/htdocs/core/boxes/box_produits.php index db7710e960b47..850ddf6a3efb1 100644 --- a/htdocs/core/boxes/box_produits.php +++ b/htdocs/core/boxes/box_produits.php @@ -65,7 +65,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf, $hookmanager; diff --git a/htdocs/core/boxes/box_produits_alerte_stock.php b/htdocs/core/boxes/box_produits_alerte_stock.php index f2a18d04ccccd..d3ff08d49b83d 100644 --- a/htdocs/core/boxes/box_produits_alerte_stock.php +++ b/htdocs/core/boxes/box_produits_alerte_stock.php @@ -67,7 +67,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf, $hookmanager; diff --git a/htdocs/core/boxes/box_project.php b/htdocs/core/boxes/box_project.php index 3feaa9f666e99..70a237e4a4083 100644 --- a/htdocs/core/boxes/box_project.php +++ b/htdocs/core/boxes/box_project.php @@ -67,7 +67,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_project_opportunities.php b/htdocs/core/boxes/box_project_opportunities.php index 35daf6083dad1..0bbfd696bdbf0 100644 --- a/htdocs/core/boxes/box_project_opportunities.php +++ b/htdocs/core/boxes/box_project_opportunities.php @@ -68,7 +68,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; diff --git a/htdocs/core/boxes/box_propales.php b/htdocs/core/boxes/box_propales.php index 09b12577f48b1..dd92282bf3282 100644 --- a/htdocs/core/boxes/box_propales.php +++ b/htdocs/core/boxes/box_propales.php @@ -63,7 +63,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_prospect.php b/htdocs/core/boxes/box_prospect.php index dd90e6f191015..3490d048fb623 100644 --- a/htdocs/core/boxes/box_prospect.php +++ b/htdocs/core/boxes/box_prospect.php @@ -71,7 +71,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $hookmanager; diff --git a/htdocs/core/boxes/box_scheduled_jobs.php b/htdocs/core/boxes/box_scheduled_jobs.php index cf1d6f683fe98..edda78c6ce4c0 100644 --- a/htdocs/core/boxes/box_scheduled_jobs.php +++ b/htdocs/core/boxes/box_scheduled_jobs.php @@ -62,7 +62,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf, $form; diff --git a/htdocs/core/boxes/box_services_contracts.php b/htdocs/core/boxes/box_services_contracts.php index 172e26153fab5..564cdd484a23c 100644 --- a/htdocs/core/boxes/box_services_contracts.php +++ b/htdocs/core/boxes/box_services_contracts.php @@ -62,7 +62,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_services_expired.php b/htdocs/core/boxes/box_services_expired.php index 13ed4b8d54daa..aee632ff98d32 100644 --- a/htdocs/core/boxes/box_services_expired.php +++ b/htdocs/core/boxes/box_services_expired.php @@ -59,7 +59,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs, $conf; diff --git a/htdocs/core/boxes/box_shipments.php b/htdocs/core/boxes/box_shipments.php index a2d54f3e94820..53027948a40b0 100644 --- a/htdocs/core/boxes/box_shipments.php +++ b/htdocs/core/boxes/box_shipments.php @@ -62,7 +62,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $user, $langs; $langs->loadLangs(array('orders', 'sendings')); diff --git a/htdocs/core/boxes/box_supplier_orders.php b/htdocs/core/boxes/box_supplier_orders.php index 1c6c9475c0960..43ad560d5b930 100644 --- a/htdocs/core/boxes/box_supplier_orders.php +++ b/htdocs/core/boxes/box_supplier_orders.php @@ -60,7 +60,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; $langs->load("boxes"); diff --git a/htdocs/core/boxes/box_supplier_orders_awaiting_reception.php b/htdocs/core/boxes/box_supplier_orders_awaiting_reception.php index 2b89a8ed4364d..43c84e9d50797 100644 --- a/htdocs/core/boxes/box_supplier_orders_awaiting_reception.php +++ b/htdocs/core/boxes/box_supplier_orders_awaiting_reception.php @@ -57,7 +57,7 @@ public function __construct($db, $param) * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; $langs->loadLangs(array("boxes", "sendings", "orders")); diff --git a/htdocs/core/boxes/box_task.php b/htdocs/core/boxes/box_task.php index 472f2116d220c..3d9a6bc0a396d 100644 --- a/htdocs/core/boxes/box_task.php +++ b/htdocs/core/boxes/box_task.php @@ -67,7 +67,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/box_validated_projects.php b/htdocs/core/boxes/box_validated_projects.php index 6c4ed5129051a..04369e6a990e8 100644 --- a/htdocs/core/boxes/box_validated_projects.php +++ b/htdocs/core/boxes/box_validated_projects.php @@ -73,7 +73,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $conf, $user, $langs; diff --git a/htdocs/core/boxes/modules_boxes.php b/htdocs/core/boxes/modules_boxes.php index 355d087e83aea..6afcb8f51bc59 100644 --- a/htdocs/core/boxes/modules_boxes.php +++ b/htdocs/core/boxes/modules_boxes.php @@ -188,7 +188,7 @@ public function __construct($db, $param = '') * @param int $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { // Must be implemented in derived classes $msg = get_class($this)."::".__FUNCTION__." not implemented"; @@ -253,7 +253,7 @@ public function fetch($rowid) * @param int<0,1> $nooutput No print, only return string * @return string */ - public function showBox($head = null, $contents = null, $nooutput = 0) + public function showBox(?array $head = null,?array $contents = null,int $nooutput = 0) { global $langs, $user, $conf; diff --git a/htdocs/core/class/commonnumrefgenerator.class.php b/htdocs/core/class/commonnumrefgenerator.class.php index 24e7991c19a6f..13ae96352d7c4 100644 --- a/htdocs/core/class/commonnumrefgenerator.class.php +++ b/htdocs/core/class/commonnumrefgenerator.class.php @@ -93,7 +93,7 @@ abstract class CommonNumRefGenerator * @deprecated Use getName() instead * @see getName() */ - public function getNom($langs) + public function getNom(Translate $langs): string { return $this->getName($langs); } @@ -103,7 +103,7 @@ public function getNom($langs) * @param Translate $langs Object langs * @return string Model name */ - public function getName($langs) + public function getName(Translate $langs): string { return empty($this->name) ? get_class($this) : $this->name; } @@ -113,7 +113,7 @@ public function getName($langs) * * @return boolean true if module can be used */ - public function isEnabled() + public function isEnabled(): bool { return true; } @@ -124,7 +124,7 @@ public function isEnabled() * @param Translate $langs Language * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("NoDescription"); } @@ -136,7 +136,7 @@ public function info($langs) * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(CommonObject $object): bool { return true; } @@ -146,7 +146,7 @@ public function canBeActivated($object) * * @return string Valeur */ - public function getVersion() + public function getVersion(): string { global $langs; $langs->load("admin"); diff --git a/htdocs/core/modules/asset/doc/doc_generic_asset_odt.modules.php b/htdocs/core/modules/asset/doc/doc_generic_asset_odt.modules.php index 92792da10e8fc..6726d8d5f02e2 100644 --- a/htdocs/core/modules/asset/doc/doc_generic_asset_odt.modules.php +++ b/htdocs/core/modules/asset/doc/doc_generic_asset_odt.modules.php @@ -101,7 +101,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/asset/mod_asset_advanced.php b/htdocs/core/modules/asset/mod_asset_advanced.php index 969abdefbe94b..6b3932128b85c 100644 --- a/htdocs/core/modules/asset/mod_asset_advanced.php +++ b/htdocs/core/modules/asset/mod_asset_advanced.php @@ -57,7 +57,7 @@ class mod_asset_advanced extends ModeleNumRefAsset * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/asset/mod_asset_standard.php b/htdocs/core/modules/asset/mod_asset_standard.php index 9e6f8e128ea77..3b47c110db0d9 100644 --- a/htdocs/core/modules/asset/mod_asset_standard.php +++ b/htdocs/core/modules/asset/mod_asset_standard.php @@ -60,7 +60,7 @@ class mod_asset_standard extends ModeleNumRefAsset * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -85,7 +85,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/barcode/doc/phpbarcode.modules.php b/htdocs/core/modules/barcode/doc/phpbarcode.modules.php index b066f2a460c39..d3632e63a2c98 100644 --- a/htdocs/core/modules/barcode/doc/phpbarcode.modules.php +++ b/htdocs/core/modules/barcode/doc/phpbarcode.modules.php @@ -62,7 +62,7 @@ public function isEnabled() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; @@ -79,7 +79,7 @@ public function info($langs) * @param Object $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $langs; diff --git a/htdocs/core/modules/barcode/doc/tcpdfbarcode.modules.php b/htdocs/core/modules/barcode/doc/tcpdfbarcode.modules.php index e93e8dfee231d..ea3dc05974574 100644 --- a/htdocs/core/modules/barcode/doc/tcpdfbarcode.modules.php +++ b/htdocs/core/modules/barcode/doc/tcpdfbarcode.modules.php @@ -51,7 +51,7 @@ class modTcpdfbarcode extends ModeleBarCode * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { return 'TCPDF-barcode'; } @@ -73,7 +73,7 @@ public function isEnabled() * @param Object $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { return true; } diff --git a/htdocs/core/modules/barcode/mod_barcode_product_standard.php b/htdocs/core/modules/barcode/mod_barcode_product_standard.php index ca5c3b7a3e046..742b4657c4960 100644 --- a/htdocs/core/modules/barcode/mod_barcode_product_standard.php +++ b/htdocs/core/modules/barcode/mod_barcode_product_standard.php @@ -66,7 +66,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $mc; global $form; diff --git a/htdocs/core/modules/barcode/mod_barcode_thirdparty_standard.php b/htdocs/core/modules/barcode/mod_barcode_thirdparty_standard.php index cf3cb3135a4ae..2ea6e2a4d177e 100644 --- a/htdocs/core/modules/barcode/mod_barcode_thirdparty_standard.php +++ b/htdocs/core/modules/barcode/mod_barcode_thirdparty_standard.php @@ -102,7 +102,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $mc; global $form; diff --git a/htdocs/core/modules/barcode/modules_barcode.class.php b/htdocs/core/modules/barcode/modules_barcode.class.php index 0c1caa9cd3e64..43710074ab76b 100644 --- a/htdocs/core/modules/barcode/modules_barcode.class.php +++ b/htdocs/core/modules/barcode/modules_barcode.class.php @@ -57,7 +57,7 @@ public function isEnabled() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - abstract public function info($langs); + abstract public function info(Translate $langs): string; /** diff --git a/htdocs/core/modules/bom/doc/doc_generic_bom_odt.modules.php b/htdocs/core/modules/bom/doc/doc_generic_bom_odt.modules.php index f8a99d51d4061..2ec70deade975 100644 --- a/htdocs/core/modules/bom/doc/doc_generic_bom_odt.modules.php +++ b/htdocs/core/modules/bom/doc/doc_generic_bom_odt.modules.php @@ -96,7 +96,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/bom/mod_bom_advanced.php b/htdocs/core/modules/bom/mod_bom_advanced.php index bc3ef9a72ff20..4bf258c41a1a7 100644 --- a/htdocs/core/modules/bom/mod_bom_advanced.php +++ b/htdocs/core/modules/bom/mod_bom_advanced.php @@ -58,7 +58,7 @@ class mod_bom_advanced extends ModeleNumRefBoms * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/bom/mod_bom_standard.php b/htdocs/core/modules/bom/mod_bom_standard.php index d4c75c1068deb..b4818bbd8705b 100644 --- a/htdocs/core/modules/bom/mod_bom_standard.php +++ b/htdocs/core/modules/bom/mod_bom_standard.php @@ -59,7 +59,7 @@ class mod_bom_standard extends ModeleNumRefBoms * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -84,7 +84,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/cheque/mod_chequereceipt_mint.php b/htdocs/core/modules/cheque/mod_chequereceipt_mint.php index 7facfcedfbbed..4439b8aa447cf 100644 --- a/htdocs/core/modules/cheque/mod_chequereceipt_mint.php +++ b/htdocs/core/modules/cheque/mod_chequereceipt_mint.php @@ -56,7 +56,7 @@ class mod_chequereceipt_mint extends ModeleNumRefChequeReceipts * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -81,7 +81,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/cheque/mod_chequereceipt_thyme.php b/htdocs/core/modules/cheque/mod_chequereceipt_thyme.php index 848b3ccbace30..863a7acfdeddc 100644 --- a/htdocs/core/modules/cheque/mod_chequereceipt_thyme.php +++ b/htdocs/core/modules/cheque/mod_chequereceipt_thyme.php @@ -52,7 +52,7 @@ class mod_chequereceipt_thyme extends ModeleNumRefChequeReceipts * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/commande/doc/doc_generic_order_odt.modules.php b/htdocs/core/modules/commande/doc/doc_generic_order_odt.modules.php index 41ff51ed49d86..049c1f4f055c8 100644 --- a/htdocs/core/modules/commande/doc/doc_generic_order_odt.modules.php +++ b/htdocs/core/modules/commande/doc/doc_generic_order_odt.modules.php @@ -104,7 +104,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/commande/mod_commande_marbre.php b/htdocs/core/modules/commande/mod_commande_marbre.php index 34ad5e13b4c8a..d8fd520135c35 100644 --- a/htdocs/core/modules/commande/mod_commande_marbre.php +++ b/htdocs/core/modules/commande/mod_commande_marbre.php @@ -71,7 +71,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -96,7 +96,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/commande/mod_commande_saphir.php b/htdocs/core/modules/commande/mod_commande_saphir.php index bb0603ef0f0fb..472058d98f58a 100644 --- a/htdocs/core/modules/commande/mod_commande_saphir.php +++ b/htdocs/core/modules/commande/mod_commande_saphir.php @@ -58,7 +58,7 @@ class mod_commande_saphir extends ModeleNumRefCommandes * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/contract/doc/doc_generic_contract_odt.modules.php b/htdocs/core/modules/contract/doc/doc_generic_contract_odt.modules.php index 27a295e61e49d..f4f9f45255ba0 100644 --- a/htdocs/core/modules/contract/doc/doc_generic_contract_odt.modules.php +++ b/htdocs/core/modules/contract/doc/doc_generic_contract_odt.modules.php @@ -101,7 +101,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/contract/mod_contract_magre.php b/htdocs/core/modules/contract/mod_contract_magre.php index e872357c63cfc..e0db1e78766c1 100644 --- a/htdocs/core/modules/contract/mod_contract_magre.php +++ b/htdocs/core/modules/contract/mod_contract_magre.php @@ -50,7 +50,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/contract/mod_contract_olive.php b/htdocs/core/modules/contract/mod_contract_olive.php index 3ebb48d59e8a5..cf7b9b089fbbd 100644 --- a/htdocs/core/modules/contract/mod_contract_olive.php +++ b/htdocs/core/modules/contract/mod_contract_olive.php @@ -56,7 +56,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; diff --git a/htdocs/core/modules/contract/mod_contract_serpis.php b/htdocs/core/modules/contract/mod_contract_serpis.php index 10f93b547e1c1..6aa845e27c2c7 100644 --- a/htdocs/core/modules/contract/mod_contract_serpis.php +++ b/htdocs/core/modules/contract/mod_contract_serpis.php @@ -55,7 +55,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -79,7 +79,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/delivery/mod_delivery_jade.php b/htdocs/core/modules/delivery/mod_delivery_jade.php index 50cd56bda1379..b50b55ce23a69 100644 --- a/htdocs/core/modules/delivery/mod_delivery_jade.php +++ b/htdocs/core/modules/delivery/mod_delivery_jade.php @@ -71,7 +71,7 @@ class mod_delivery_jade extends ModeleNumRefDeliveryOrder * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -94,7 +94,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $langs, $conf, $db; diff --git a/htdocs/core/modules/delivery/mod_delivery_saphir.php b/htdocs/core/modules/delivery/mod_delivery_saphir.php index f09c8764d7ae4..e97095ed7788e 100644 --- a/htdocs/core/modules/delivery/mod_delivery_saphir.php +++ b/htdocs/core/modules/delivery/mod_delivery_saphir.php @@ -63,7 +63,7 @@ class mod_delivery_saphir extends ModeleNumRefDeliveryOrder * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/expedition/doc/doc_generic_shipment_odt.modules.php b/htdocs/core/modules/expedition/doc/doc_generic_shipment_odt.modules.php index 5cbba7e69f043..98c818275453e 100644 --- a/htdocs/core/modules/expedition/doc/doc_generic_shipment_odt.modules.php +++ b/htdocs/core/modules/expedition/doc/doc_generic_shipment_odt.modules.php @@ -104,7 +104,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/expedition/mod_expedition_ribera.php b/htdocs/core/modules/expedition/mod_expedition_ribera.php index a6394c0df169c..d5bf020a23e94 100644 --- a/htdocs/core/modules/expedition/mod_expedition_ribera.php +++ b/htdocs/core/modules/expedition/mod_expedition_ribera.php @@ -60,7 +60,7 @@ class mod_expedition_ribera extends ModelNumRefExpedition * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/expedition/mod_expedition_safor.php b/htdocs/core/modules/expedition/mod_expedition_safor.php index 6ddd5231a67bf..41020b78978f7 100644 --- a/htdocs/core/modules/expedition/mod_expedition_safor.php +++ b/htdocs/core/modules/expedition/mod_expedition_safor.php @@ -65,7 +65,7 @@ class mod_expedition_safor extends ModelNumRefExpedition * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -89,7 +89,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/expensereport/mod_expensereport_jade.php b/htdocs/core/modules/expensereport/mod_expensereport_jade.php index 886f671a9cf8b..ee7e184291633 100644 --- a/htdocs/core/modules/expensereport/mod_expensereport_jade.php +++ b/htdocs/core/modules/expensereport/mod_expensereport_jade.php @@ -65,7 +65,7 @@ class mod_expensereport_jade extends ModeleNumRefExpenseReport * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -90,7 +90,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/expensereport/mod_expensereport_sand.php b/htdocs/core/modules/expensereport/mod_expensereport_sand.php index 6f4f091f90ef9..3fc4707e3df93 100644 --- a/htdocs/core/modules/expensereport/mod_expensereport_sand.php +++ b/htdocs/core/modules/expensereport/mod_expensereport_sand.php @@ -62,7 +62,7 @@ class mod_expensereport_sand extends ModeleNumRefExpenseReport * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/facture/doc/doc_generic_invoice_odt.modules.php b/htdocs/core/modules/facture/doc/doc_generic_invoice_odt.modules.php index f4435c982ed84..b37946efc7106 100644 --- a/htdocs/core/modules/facture/doc/doc_generic_invoice_odt.modules.php +++ b/htdocs/core/modules/facture/doc/doc_generic_invoice_odt.modules.php @@ -103,7 +103,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/facture/mod_facture_mars.php b/htdocs/core/modules/facture/mod_facture_mars.php index bef545ffe48d1..4afb4617196cf 100644 --- a/htdocs/core/modules/facture/mod_facture_mars.php +++ b/htdocs/core/modules/facture/mod_facture_mars.php @@ -77,7 +77,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; $langs->load("bills"); @@ -101,7 +101,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $langs, $conf, $db; diff --git a/htdocs/core/modules/facture/mod_facture_mercure.php b/htdocs/core/modules/facture/mod_facture_mercure.php index 3d3a93e8154b4..703d56f03a579 100644 --- a/htdocs/core/modules/facture/mod_facture_mercure.php +++ b/htdocs/core/modules/facture/mod_facture_mercure.php @@ -55,7 +55,7 @@ class mod_facture_mercure extends ModeleNumRefFactures * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/facture/mod_facture_terre.php b/htdocs/core/modules/facture/mod_facture_terre.php index 38866d581cc4f..027e2d6926e6e 100644 --- a/htdocs/core/modules/facture/mod_facture_terre.php +++ b/htdocs/core/modules/facture/mod_facture_terre.php @@ -93,7 +93,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; $langs->load("bills"); @@ -117,7 +117,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $langs, $conf, $db; diff --git a/htdocs/core/modules/fichinter/mod_arctic.php b/htdocs/core/modules/fichinter/mod_arctic.php index a23fe52bbf63c..310adf70737cf 100644 --- a/htdocs/core/modules/fichinter/mod_arctic.php +++ b/htdocs/core/modules/fichinter/mod_arctic.php @@ -64,7 +64,7 @@ class mod_arctic extends ModeleNumRefFicheinter * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/fichinter/mod_pacific.php b/htdocs/core/modules/fichinter/mod_pacific.php index f6eaa326c7560..21f9a696ca79d 100644 --- a/htdocs/core/modules/fichinter/mod_pacific.php +++ b/htdocs/core/modules/fichinter/mod_pacific.php @@ -67,7 +67,7 @@ class mod_pacific extends ModeleNumRefFicheinter * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -90,7 +90,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $langs, $conf, $db; diff --git a/htdocs/core/modules/holiday/mod_holiday_immaculate.php b/htdocs/core/modules/holiday/mod_holiday_immaculate.php index c8ed8ec59965d..1af98de656852 100644 --- a/htdocs/core/modules/holiday/mod_holiday_immaculate.php +++ b/htdocs/core/modules/holiday/mod_holiday_immaculate.php @@ -50,7 +50,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/holiday/mod_holiday_madonna.php b/htdocs/core/modules/holiday/mod_holiday_madonna.php index 9f10ed28eef72..e1fbf024e31ee 100644 --- a/htdocs/core/modules/holiday/mod_holiday_madonna.php +++ b/htdocs/core/modules/holiday/mod_holiday_madonna.php @@ -56,7 +56,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -80,7 +80,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/hrm/mod_evaluation_advanced.php b/htdocs/core/modules/hrm/mod_evaluation_advanced.php index 434512e1cb115..9960031384510 100644 --- a/htdocs/core/modules/hrm/mod_evaluation_advanced.php +++ b/htdocs/core/modules/hrm/mod_evaluation_advanced.php @@ -58,7 +58,7 @@ class mod_evaluation_advanced extends ModeleNumRefEvaluation * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/hrm/mod_evaluation_standard.php b/htdocs/core/modules/hrm/mod_evaluation_standard.php index 70a65e9991366..2d69564cbc3c4 100644 --- a/htdocs/core/modules/hrm/mod_evaluation_standard.php +++ b/htdocs/core/modules/hrm/mod_evaluation_standard.php @@ -60,7 +60,7 @@ class mod_evaluation_standard extends ModeleNumRefEvaluation * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -85,7 +85,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/mailings/modules_mailings.php b/htdocs/core/modules/mailings/modules_mailings.php index e45bd655ef3f6..0839962842ca3 100644 --- a/htdocs/core/modules/mailings/modules_mailings.php +++ b/htdocs/core/modules/mailings/modules_mailings.php @@ -84,7 +84,7 @@ class MailingTargets // This can't be abstract as it is used for some method * * @param DoliDB $db Database handler */ - public function __construct($db) + public function __construct(DoliDB $db) { $this->db = $db; } @@ -94,7 +94,7 @@ public function __construct($db) * * @return string Return translation of module label. Try translation of $this->name then translation of 'MailingModuleDesc'.$this->name, or $this->desc if not found */ - public function getDesc() + public function getDesc(): string { global $langs, $form; @@ -121,7 +121,7 @@ public function getDesc() * * @return integer Example */ - public function getNbOfRecords() + public function getNbOfRecords(): int { return 0; } @@ -132,7 +132,7 @@ public function getNbOfRecords() * @param string $sql Sql request to count * @return int|string Nb of recipient, or <0 if error, or '' if NA */ - public function getNbOfRecipients($sql) + public function getNbOfRecipients(string $sql) { $result = $this->db->query($sql); if ($result) { @@ -153,7 +153,7 @@ public function getNbOfRecipients($sql) * * @return string Retourne zone select */ - public function formFilter() + public function formFilter(): string { return ''; } @@ -165,7 +165,7 @@ public function formFilter() * @param int $mailing_id Id of emailing * @return int Return integer < 0 si erreur, nb destinataires si ok */ - public function update_nb($mailing_id) + public function update_nb(int $mailing_id): int { // phpcs:enable // Mise a jour nombre de destinataire dans table des mailings @@ -196,7 +196,7 @@ public function update_nb($mailing_id) * @param array $cibles Array with targets * @return int Return integer < 0 if error, nb added if OK */ - public function addTargetsToDatabase($mailing_id, $cibles) + public function addTargetsToDatabase(int $mailing_id,array $cibles): int { global $conf; @@ -288,7 +288,7 @@ public function addTargetsToDatabase($mailing_id, $cibles) * @param int $mailing_id Id of emailing * @return void */ - public function clear_target($mailing_id) + public function clear_target(int $mailing_id): void { // phpcs:enable $sql = "DELETE FROM ".MAIN_DB_PREFIX."mailing_cibles"; @@ -309,7 +309,7 @@ public function clear_target($mailing_id) * @param ?array $forcedir null=All default directories. This parameter is used by modulebuilder module only. * @return array Array list of widgets */ - public static function getEmailingSelectorsList($forcedir = null) + public static function getEmailingSelectorsList(?array $forcedir = null): array { global $langs, $db; @@ -442,7 +442,7 @@ public static function getEmailingSelectorsList($forcedir = null) * * @return string[] Array with SQL requests */ - public function getSqlArrayForStats() + public function getSqlArrayForStats(): array { // Needs to be implemented in child class $msg = get_class($this)."::".__FUNCTION__." not implemented"; @@ -457,7 +457,7 @@ public function getSqlArrayForStats() * @param int $mailing_id Id of emailing * @return int Return integer < 0 on error, count of added when ok */ - public function add_to_target($mailing_id) + public function add_to_target(int $mailing_id): int { // phpcs:enable // Needs to be implemented in child class diff --git a/htdocs/core/modules/member/doc/doc_generic_member_odt.class.php b/htdocs/core/modules/member/doc/doc_generic_member_odt.class.php index 6834dcfd7e768..a2a0c8feda59d 100644 --- a/htdocs/core/modules/member/doc/doc_generic_member_odt.class.php +++ b/htdocs/core/modules/member/doc/doc_generic_member_odt.class.php @@ -100,7 +100,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $langs; diff --git a/htdocs/core/modules/member/mod_member_advanced.php b/htdocs/core/modules/member/mod_member_advanced.php index a158c9ef21d2f..cc8eaaa94e007 100644 --- a/htdocs/core/modules/member/mod_member_advanced.php +++ b/htdocs/core/modules/member/mod_member_advanced.php @@ -57,7 +57,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("AdvancedNumRefModelDesc", $this->prefix); @@ -82,7 +82,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/member/mod_member_simple.php b/htdocs/core/modules/member/mod_member_simple.php index e35531f326542..dcf2e2668a6f2 100644 --- a/htdocs/core/modules/member/mod_member_simple.php +++ b/htdocs/core/modules/member/mod_member_simple.php @@ -57,7 +57,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleRefNumRefModelDesc"); @@ -82,7 +82,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/mrp/doc/doc_generic_mo_odt.modules.php b/htdocs/core/modules/mrp/doc/doc_generic_mo_odt.modules.php index ff22719c4ab72..7dd04afdfad5f 100644 --- a/htdocs/core/modules/mrp/doc/doc_generic_mo_odt.modules.php +++ b/htdocs/core/modules/mrp/doc/doc_generic_mo_odt.modules.php @@ -103,7 +103,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/mrp/mod_mo_advanced.php b/htdocs/core/modules/mrp/mod_mo_advanced.php index 82286280b8eeb..5df101af6af2d 100644 --- a/htdocs/core/modules/mrp/mod_mo_advanced.php +++ b/htdocs/core/modules/mrp/mod_mo_advanced.php @@ -59,7 +59,7 @@ class mod_mo_advanced extends ModeleNumRefMos * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/mrp/mod_mo_standard.php b/htdocs/core/modules/mrp/mod_mo_standard.php index 039ec18eb61fc..6b03b764d6353 100644 --- a/htdocs/core/modules/mrp/mod_mo_standard.php +++ b/htdocs/core/modules/mrp/mod_mo_standard.php @@ -59,7 +59,7 @@ class mod_mo_standard extends ModeleNumRefMos * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -84,7 +84,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/payment/mod_payment_ant.php b/htdocs/core/modules/payment/mod_payment_ant.php index 5e3f0a4b37da9..135514f5f0b89 100644 --- a/htdocs/core/modules/payment/mod_payment_ant.php +++ b/htdocs/core/modules/payment/mod_payment_ant.php @@ -61,7 +61,7 @@ class mod_payment_ant extends ModeleNumRefPayments * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/payment/mod_payment_cicada.php b/htdocs/core/modules/payment/mod_payment_cicada.php index e9e3a41494990..fdddc4f223830 100644 --- a/htdocs/core/modules/payment/mod_payment_cicada.php +++ b/htdocs/core/modules/payment/mod_payment_cicada.php @@ -66,7 +66,7 @@ class mod_payment_cicada extends ModeleNumRefPayments * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -91,7 +91,7 @@ public function getExample() * @param Object $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/product/doc/doc_generic_product_odt.modules.php b/htdocs/core/modules/product/doc/doc_generic_product_odt.modules.php index e6f0d524747e2..1dec3be94fd77 100644 --- a/htdocs/core/modules/product/doc/doc_generic_product_odt.modules.php +++ b/htdocs/core/modules/product/doc/doc_generic_product_odt.modules.php @@ -101,7 +101,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/product/mod_codeproduct_elephant.php b/htdocs/core/modules/product/mod_codeproduct_elephant.php index da6475dc40121..8080bf6ae2f8a 100644 --- a/htdocs/core/modules/product/mod_codeproduct_elephant.php +++ b/htdocs/core/modules/product/mod_codeproduct_elephant.php @@ -72,7 +72,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $mc; global $form; diff --git a/htdocs/core/modules/product/mod_codeproduct_leopard.php b/htdocs/core/modules/product/mod_codeproduct_leopard.php index cc954cace2269..45a1b3584afe4 100644 --- a/htdocs/core/modules/product/mod_codeproduct_leopard.php +++ b/htdocs/core/modules/product/mod_codeproduct_leopard.php @@ -63,7 +63,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { $langs->load("companies"); return $langs->trans("LeopardNumRefModelDesc"); diff --git a/htdocs/core/modules/product_batch/mod_lot_advanced.php b/htdocs/core/modules/product_batch/mod_lot_advanced.php index d07ce3fd9e35a..2025144343a57 100644 --- a/htdocs/core/modules/product_batch/mod_lot_advanced.php +++ b/htdocs/core/modules/product_batch/mod_lot_advanced.php @@ -59,7 +59,7 @@ class mod_lot_advanced extends ModeleNumRefBatch * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs, $db; diff --git a/htdocs/core/modules/product_batch/mod_lot_free.php b/htdocs/core/modules/product_batch/mod_lot_free.php index 4e6b356f15f0a..b8a4bd52168fb 100644 --- a/htdocs/core/modules/product_batch/mod_lot_free.php +++ b/htdocs/core/modules/product_batch/mod_lot_free.php @@ -63,7 +63,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; $langs->load("companies"); diff --git a/htdocs/core/modules/product_batch/mod_lot_standard.php b/htdocs/core/modules/product_batch/mod_lot_standard.php index 209a811120c53..501231e54d624 100644 --- a/htdocs/core/modules/product_batch/mod_lot_standard.php +++ b/htdocs/core/modules/product_batch/mod_lot_standard.php @@ -60,7 +60,7 @@ class mod_lot_standard extends ModeleNumRefBatch * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -85,7 +85,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/product_batch/mod_sn_advanced.php b/htdocs/core/modules/product_batch/mod_sn_advanced.php index 18a33833ed3d1..c3ad78f85dc34 100644 --- a/htdocs/core/modules/product_batch/mod_sn_advanced.php +++ b/htdocs/core/modules/product_batch/mod_sn_advanced.php @@ -59,7 +59,7 @@ class mod_sn_advanced extends ModeleNumRefBatch * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs, $db; diff --git a/htdocs/core/modules/product_batch/mod_sn_free.php b/htdocs/core/modules/product_batch/mod_sn_free.php index f241f106cf75b..81d395408a397 100644 --- a/htdocs/core/modules/product_batch/mod_sn_free.php +++ b/htdocs/core/modules/product_batch/mod_sn_free.php @@ -62,7 +62,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; $langs->load("companies"); diff --git a/htdocs/core/modules/product_batch/mod_sn_standard.php b/htdocs/core/modules/product_batch/mod_sn_standard.php index b59e77a3e94c1..882b7deff6849 100644 --- a/htdocs/core/modules/product_batch/mod_sn_standard.php +++ b/htdocs/core/modules/product_batch/mod_sn_standard.php @@ -60,7 +60,7 @@ class mod_sn_standard extends ModeleNumRefBatch * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -85,7 +85,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/project/doc/doc_generic_project_odt.modules.php b/htdocs/core/modules/project/doc/doc_generic_project_odt.modules.php index 63f17a6a644db..c125d0ac2d465 100644 --- a/htdocs/core/modules/project/doc/doc_generic_project_odt.modules.php +++ b/htdocs/core/modules/project/doc/doc_generic_project_odt.modules.php @@ -393,7 +393,7 @@ public function get_substitutionarray_task_file($file, $outputlangs) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/project/mod_project_simple.php b/htdocs/core/modules/project/mod_project_simple.php index d6ad1781294a9..2202e7474c601 100644 --- a/htdocs/core/modules/project/mod_project_simple.php +++ b/htdocs/core/modules/project/mod_project_simple.php @@ -68,7 +68,7 @@ class mod_project_simple extends ModeleNumRefProjects * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -93,7 +93,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/project/mod_project_universal.php b/htdocs/core/modules/project/mod_project_universal.php index 4132956f86ef5..b983160cf7495 100644 --- a/htdocs/core/modules/project/mod_project_universal.php +++ b/htdocs/core/modules/project/mod_project_universal.php @@ -67,7 +67,7 @@ class mod_project_universal extends ModeleNumRefProjects * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; diff --git a/htdocs/core/modules/project/task/doc/doc_generic_task_odt.modules.php b/htdocs/core/modules/project/task/doc/doc_generic_task_odt.modules.php index a20c933500a73..caf61c32667f7 100644 --- a/htdocs/core/modules/project/task/doc/doc_generic_task_odt.modules.php +++ b/htdocs/core/modules/project/task/doc/doc_generic_task_odt.modules.php @@ -359,7 +359,7 @@ public function get_substitutionarray_task_file($file, $outputlangs) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/project/task/mod_task_simple.php b/htdocs/core/modules/project/task/mod_task_simple.php index 3647b6104fa8f..9ae024e87b1a1 100644 --- a/htdocs/core/modules/project/task/mod_task_simple.php +++ b/htdocs/core/modules/project/task/mod_task_simple.php @@ -68,7 +68,7 @@ class mod_task_simple extends ModeleNumRefTask * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -93,7 +93,7 @@ public function getExample() * @param Object $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/project/task/mod_task_universal.php b/htdocs/core/modules/project/task/mod_task_universal.php index 4fcb310c9416e..f6d170c3c1373 100644 --- a/htdocs/core/modules/project/task/mod_task_universal.php +++ b/htdocs/core/modules/project/task/mod_task_universal.php @@ -62,7 +62,7 @@ class mod_task_universal extends ModeleNumRefTask * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/propale/doc/doc_generic_proposal_odt.modules.php b/htdocs/core/modules/propale/doc/doc_generic_proposal_odt.modules.php index 86266b76b1673..315a22eb553f6 100644 --- a/htdocs/core/modules/propale/doc/doc_generic_proposal_odt.modules.php +++ b/htdocs/core/modules/propale/doc/doc_generic_proposal_odt.modules.php @@ -103,7 +103,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $langs; diff --git a/htdocs/core/modules/propale/mod_propale_marbre.php b/htdocs/core/modules/propale/mod_propale_marbre.php index 4d2042dfa4037..541f63846b0da 100644 --- a/htdocs/core/modules/propale/mod_propale_marbre.php +++ b/htdocs/core/modules/propale/mod_propale_marbre.php @@ -68,7 +68,7 @@ class mod_propale_marbre extends ModeleNumRefPropales * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -93,7 +93,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/propale/mod_propale_saphir.php b/htdocs/core/modules/propale/mod_propale_saphir.php index 93ebf4f82980e..e9fb28ebbfe46 100644 --- a/htdocs/core/modules/propale/mod_propale_saphir.php +++ b/htdocs/core/modules/propale/mod_propale_saphir.php @@ -65,7 +65,7 @@ class mod_propale_saphir extends ModeleNumRefPropales * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs, $db; diff --git a/htdocs/core/modules/reception/doc/doc_generic_reception_odt.modules.php b/htdocs/core/modules/reception/doc/doc_generic_reception_odt.modules.php index bf7247fdde820..273d21be592ac 100644 --- a/htdocs/core/modules/reception/doc/doc_generic_reception_odt.modules.php +++ b/htdocs/core/modules/reception/doc/doc_generic_reception_odt.modules.php @@ -98,7 +98,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/reception/mod_reception_beryl.php b/htdocs/core/modules/reception/mod_reception_beryl.php index ef60d5747e38d..941dddd1ac922 100644 --- a/htdocs/core/modules/reception/mod_reception_beryl.php +++ b/htdocs/core/modules/reception/mod_reception_beryl.php @@ -48,7 +48,7 @@ class mod_reception_beryl extends ModelNumRefReception * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -72,7 +72,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/reception/mod_reception_moonstone.php b/htdocs/core/modules/reception/mod_reception_moonstone.php index a482366fc36b8..d59e4fa1c8035 100644 --- a/htdocs/core/modules/reception/mod_reception_moonstone.php +++ b/htdocs/core/modules/reception/mod_reception_moonstone.php @@ -44,7 +44,7 @@ class mod_reception_moonstone extends ModelNumRefReception * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/societe/doc/doc_generic_odt.modules.php b/htdocs/core/modules/societe/doc/doc_generic_odt.modules.php index b95f709c55a70..80ac7c8439fad 100644 --- a/htdocs/core/modules/societe/doc/doc_generic_odt.modules.php +++ b/htdocs/core/modules/societe/doc/doc_generic_odt.modules.php @@ -93,7 +93,7 @@ public function __construct($db) * @param Translate $langs Object language * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/societe/mod_codeclient_elephant.php b/htdocs/core/modules/societe/mod_codeclient_elephant.php index b01fcafb6db84..73bb5ffee7972 100644 --- a/htdocs/core/modules/societe/mod_codeclient_elephant.php +++ b/htdocs/core/modules/societe/mod_codeclient_elephant.php @@ -78,7 +78,7 @@ public function __construct($db) * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $mc; global $form; diff --git a/htdocs/core/modules/societe/mod_codeclient_leopard.php b/htdocs/core/modules/societe/mod_codeclient_leopard.php index e7509a07e3b3a..eb363b82be12f 100644 --- a/htdocs/core/modules/societe/mod_codeclient_leopard.php +++ b/htdocs/core/modules/societe/mod_codeclient_leopard.php @@ -68,7 +68,7 @@ public function __construct($db) * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { $langs->load("companies"); return $langs->trans("LeopardNumRefModelDesc"); diff --git a/htdocs/core/modules/societe/mod_codeclient_monkey.php b/htdocs/core/modules/societe/mod_codeclient_monkey.php index 5eb500d61c9e5..5856a108f8af6 100644 --- a/htdocs/core/modules/societe/mod_codeclient_monkey.php +++ b/htdocs/core/modules/societe/mod_codeclient_monkey.php @@ -67,7 +67,7 @@ public function __construct($db) * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("MonkeyNumRefModelDesc", $this->prefixcustomer, $this->prefixsupplier); } diff --git a/htdocs/core/modules/societe/mod_codecompta_aquarium.php b/htdocs/core/modules/societe/mod_codecompta_aquarium.php index 07ad702dd3682..5acaa16a40c36 100644 --- a/htdocs/core/modules/societe/mod_codecompta_aquarium.php +++ b/htdocs/core/modules/societe/mod_codecompta_aquarium.php @@ -85,7 +85,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { global $form; diff --git a/htdocs/core/modules/societe/mod_codecompta_digitaria.php b/htdocs/core/modules/societe/mod_codecompta_digitaria.php index 857cb2d9521f5..b08ec73bf3d30 100644 --- a/htdocs/core/modules/societe/mod_codecompta_digitaria.php +++ b/htdocs/core/modules/societe/mod_codecompta_digitaria.php @@ -97,7 +97,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $form; diff --git a/htdocs/core/modules/societe/mod_codecompta_panicum.php b/htdocs/core/modules/societe/mod_codecompta_panicum.php index f16bb7b7ea42f..1b752f261af3f 100644 --- a/htdocs/core/modules/societe/mod_codecompta_panicum.php +++ b/htdocs/core/modules/societe/mod_codecompta_panicum.php @@ -65,7 +65,7 @@ public function __construct() * @param Translate $langs Object langs * @return string Description of module */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("ModuleCompanyCode".$this->name); } diff --git a/htdocs/core/modules/stock/doc/doc_generic_stock_odt.modules.php b/htdocs/core/modules/stock/doc/doc_generic_stock_odt.modules.php index fbb1bc6428027..fba6e896db9b0 100644 --- a/htdocs/core/modules/stock/doc/doc_generic_stock_odt.modules.php +++ b/htdocs/core/modules/stock/doc/doc_generic_stock_odt.modules.php @@ -100,7 +100,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/stocktransfer/mod_stocktransfer_advanced.php b/htdocs/core/modules/stocktransfer/mod_stocktransfer_advanced.php index 0bc819da50b26..f775e527a354b 100644 --- a/htdocs/core/modules/stocktransfer/mod_stocktransfer_advanced.php +++ b/htdocs/core/modules/stocktransfer/mod_stocktransfer_advanced.php @@ -59,7 +59,7 @@ class mod_stocktransfer_advanced extends ModeleNumRefStockTransfer * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/stocktransfer/mod_stocktransfer_standard.php b/htdocs/core/modules/stocktransfer/mod_stocktransfer_standard.php index e2f2335cb8214..24a73bb57f90b 100644 --- a/htdocs/core/modules/stocktransfer/mod_stocktransfer_standard.php +++ b/htdocs/core/modules/stocktransfer/mod_stocktransfer_standard.php @@ -61,7 +61,7 @@ class mod_stocktransfer_standard extends ModeleNumRefStockTransfer * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -86,7 +86,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/supplier_invoice/doc/doc_generic_supplier_invoice_odt.modules.php b/htdocs/core/modules/supplier_invoice/doc/doc_generic_supplier_invoice_odt.modules.php index cd4fbe8617bbf..245d971078805 100644 --- a/htdocs/core/modules/supplier_invoice/doc/doc_generic_supplier_invoice_odt.modules.php +++ b/htdocs/core/modules/supplier_invoice/doc/doc_generic_supplier_invoice_odt.modules.php @@ -109,7 +109,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_cactus.php b/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_cactus.php index aa760b7c50da4..8de3b6d55cad2 100644 --- a/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_cactus.php +++ b/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_cactus.php @@ -80,7 +80,7 @@ class mod_facture_fournisseur_cactus extends ModeleNumRefSuppliersInvoices * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; $langs->load("bills"); @@ -105,7 +105,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_tulip.php b/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_tulip.php index bafa84d20c880..b7803748e32db 100644 --- a/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_tulip.php +++ b/htdocs/core/modules/supplier_invoice/mod_facture_fournisseur_tulip.php @@ -68,7 +68,7 @@ class mod_facture_fournisseur_tulip extends ModeleNumRefSuppliersInvoices * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/supplier_order/doc/doc_generic_supplier_order_odt.modules.php b/htdocs/core/modules/supplier_order/doc/doc_generic_supplier_order_odt.modules.php index f6407073b83f9..29d0a7e65a540 100644 --- a/htdocs/core/modules/supplier_order/doc/doc_generic_supplier_order_odt.modules.php +++ b/htdocs/core/modules/supplier_order/doc/doc_generic_supplier_order_odt.modules.php @@ -109,7 +109,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/supplier_order/mod_commande_fournisseur_muguet.php b/htdocs/core/modules/supplier_order/mod_commande_fournisseur_muguet.php index 1599ca8e222b4..0ff407f6c0b64 100644 --- a/htdocs/core/modules/supplier_order/mod_commande_fournisseur_muguet.php +++ b/htdocs/core/modules/supplier_order/mod_commande_fournisseur_muguet.php @@ -78,7 +78,7 @@ public function __construct() * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -103,7 +103,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/supplier_order/mod_commande_fournisseur_orchidee.php b/htdocs/core/modules/supplier_order/mod_commande_fournisseur_orchidee.php index 707b64505d63d..e8d8d13995ad6 100644 --- a/htdocs/core/modules/supplier_order/mod_commande_fournisseur_orchidee.php +++ b/htdocs/core/modules/supplier_order/mod_commande_fournisseur_orchidee.php @@ -64,7 +64,7 @@ class mod_commande_fournisseur_orchidee extends ModeleNumRefSuppliersOrders * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/supplier_payment/mod_supplier_payment_brodator.php b/htdocs/core/modules/supplier_payment/mod_supplier_payment_brodator.php index 628b7411ebfbf..fce80939a2d5c 100644 --- a/htdocs/core/modules/supplier_payment/mod_supplier_payment_brodator.php +++ b/htdocs/core/modules/supplier_payment/mod_supplier_payment_brodator.php @@ -62,7 +62,7 @@ class mod_supplier_payment_brodator extends ModeleNumRefSupplierPayments * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/supplier_payment/mod_supplier_payment_bronan.php b/htdocs/core/modules/supplier_payment/mod_supplier_payment_bronan.php index 8d9192cc6ff50..a0df0c7cf6a8a 100644 --- a/htdocs/core/modules/supplier_payment/mod_supplier_payment_bronan.php +++ b/htdocs/core/modules/supplier_payment/mod_supplier_payment_bronan.php @@ -66,7 +66,7 @@ class mod_supplier_payment_bronan extends ModeleNumRefSupplierPayments * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -91,7 +91,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/supplier_proposal/doc/doc_generic_supplier_proposal_odt.modules.php b/htdocs/core/modules/supplier_proposal/doc/doc_generic_supplier_proposal_odt.modules.php index a17f724a5fa0f..4f46d321e1ebd 100644 --- a/htdocs/core/modules/supplier_proposal/doc/doc_generic_supplier_proposal_odt.modules.php +++ b/htdocs/core/modules/supplier_proposal/doc/doc_generic_supplier_proposal_odt.modules.php @@ -102,7 +102,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $langs; diff --git a/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_marbre.php b/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_marbre.php index ca34cc22b9f1e..d7e137cc897c3 100644 --- a/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_marbre.php +++ b/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_marbre.php @@ -68,7 +68,7 @@ class mod_supplier_proposal_marbre extends ModeleNumRefSupplierProposal * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -93,7 +93,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_saphir.php b/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_saphir.php index 9c83c7d917647..530a13470f539 100644 --- a/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_saphir.php +++ b/htdocs/core/modules/supplier_proposal/mod_supplier_proposal_saphir.php @@ -64,7 +64,7 @@ class mod_supplier_proposal_saphir extends ModeleNumRefSupplierProposal * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/takepos/mod_takepos_ref_simple.php b/htdocs/core/modules/takepos/mod_takepos_ref_simple.php index 6d57bad5c0f92..4b0827f4f2e0f 100644 --- a/htdocs/core/modules/takepos/mod_takepos_ref_simple.php +++ b/htdocs/core/modules/takepos/mod_takepos_ref_simple.php @@ -63,7 +63,7 @@ class mod_takepos_ref_simple extends ModeleNumRefTakepos * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; @@ -90,7 +90,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if KO (there is a conflict), true if OK */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/takepos/mod_takepos_ref_universal.php b/htdocs/core/modules/takepos/mod_takepos_ref_universal.php index 75768b9f87e11..6d80b3cb0ce24 100644 --- a/htdocs/core/modules/takepos/mod_takepos_ref_universal.php +++ b/htdocs/core/modules/takepos/mod_takepos_ref_universal.php @@ -59,7 +59,7 @@ class mod_takepos_ref_universal extends ModeleNumRefTakepos * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/ticket/doc/doc_generic_ticket_odt.modules.php b/htdocs/core/modules/ticket/doc/doc_generic_ticket_odt.modules.php index c1046c3885a0e..3a05e1212f515 100644 --- a/htdocs/core/modules/ticket/doc/doc_generic_ticket_odt.modules.php +++ b/htdocs/core/modules/ticket/doc/doc_generic_ticket_odt.modules.php @@ -96,7 +96,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $langs; diff --git a/htdocs/core/modules/ticket/mod_ticket_simple.php b/htdocs/core/modules/ticket/mod_ticket_simple.php index 576da3eff839f..595ece7dc7071 100644 --- a/htdocs/core/modules/ticket/mod_ticket_simple.php +++ b/htdocs/core/modules/ticket/mod_ticket_simple.php @@ -66,7 +66,7 @@ class mod_ticket_simple extends ModeleNumRefTicket * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefModelDesc", $this->prefix); @@ -89,7 +89,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/core/modules/ticket/mod_ticket_universal.php b/htdocs/core/modules/ticket/mod_ticket_universal.php index dcac49b041acc..2aa670968e886 100644 --- a/htdocs/core/modules/ticket/mod_ticket_universal.php +++ b/htdocs/core/modules/ticket/mod_ticket_universal.php @@ -60,7 +60,7 @@ class mod_ticket_universal extends ModeleNumRefTicket * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db, $langs; diff --git a/htdocs/core/modules/user/doc/doc_generic_user_odt.modules.php b/htdocs/core/modules/user/doc/doc_generic_user_odt.modules.php index 78f9e00ca361b..ec610f1c5248b 100644 --- a/htdocs/core/modules/user/doc/doc_generic_user_odt.modules.php +++ b/htdocs/core/modules/user/doc/doc_generic_user_odt.modules.php @@ -100,7 +100,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/usergroup/doc/doc_generic_usergroup_odt.modules.php b/htdocs/core/modules/usergroup/doc/doc_generic_usergroup_odt.modules.php index da30d9a0b2148..bc1d27a55fa9d 100644 --- a/htdocs/core/modules/usergroup/doc/doc_generic_usergroup_odt.modules.php +++ b/htdocs/core/modules/usergroup/doc/doc_generic_usergroup_odt.modules.php @@ -102,7 +102,7 @@ public function __construct($db) * @param Translate $langs Lang object to use for output * @return string Description */ - public function info($langs) + public function info(Translate $langs): string { global $conf, $langs; diff --git a/htdocs/core/modules/workstation/mod_workstation_advanced.php b/htdocs/core/modules/workstation/mod_workstation_advanced.php index c67765e88d814..9a88760eecca6 100644 --- a/htdocs/core/modules/workstation/mod_workstation_advanced.php +++ b/htdocs/core/modules/workstation/mod_workstation_advanced.php @@ -59,7 +59,7 @@ class mod_workstation_advanced extends ModeleNumRefWorkstation * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs, $db; diff --git a/htdocs/core/modules/workstation/mod_workstation_standard.php b/htdocs/core/modules/workstation/mod_workstation_standard.php index 7950cd488f231..4e6790e41e39c 100644 --- a/htdocs/core/modules/workstation/mod_workstation_standard.php +++ b/htdocs/core/modules/workstation/mod_workstation_standard.php @@ -60,7 +60,7 @@ class mod_workstation_standard extends ModeleNumRefWorkstation * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $langs; return $langs->trans("SimpleNumRefNoDateModelDesc", $this->prefix); @@ -85,7 +85,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $db; diff --git a/htdocs/core/triggers/dolibarrtriggers.class.php b/htdocs/core/triggers/dolibarrtriggers.class.php index ad8bd1fa159c1..53ffba3ec7463 100644 --- a/htdocs/core/triggers/dolibarrtriggers.class.php +++ b/htdocs/core/triggers/dolibarrtriggers.class.php @@ -124,7 +124,7 @@ public function __construct(DoliDB $db) * * @return string */ - public function getName() + public function getName(): string { return $this->name; } @@ -134,7 +134,7 @@ public function getName() * * @return string */ - public function getDesc() + public function getDesc(): string { return $this->description; } @@ -144,7 +144,7 @@ public function getDesc() * * @return string Version of trigger file */ - public function getVersion() + public function getVersion(): string { global $langs; $langs->load("admin"); @@ -166,7 +166,7 @@ public function getVersion() * @param CommonObject $object Object * @return void */ - public function setErrorsFromObject(CommonObject $object) + public function setErrorsFromObject(CommonObject $object): void { if (!empty($object->error)) { $this->errors = array_merge($this->errors, array($object->error)); @@ -181,11 +181,11 @@ public function setErrorsFromObject(CommonObject $object) * All functions "runTrigger" are triggered if file is inside directory htdocs/core/triggers or htdocs/module/code/triggers (and declared) * * @param string $action Event action code - * @param Object $object Object + * @param CommonObject $object Object * @param User $user Object user * @param Translate $langs Object langs * @param conf $conf Object conf * @return int if KO: <0 || if no trigger ran: 0 || if OK: >0 */ - abstract public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf); + abstract public function runTrigger(string $action,CommonObject $object, User $user, Translate $langs, Conf $conf): int; } diff --git a/htdocs/core/triggers/interface_20_all_Logevents.class.php b/htdocs/core/triggers/interface_20_all_Logevents.class.php index b948ef9948444..605d3ab3e89ab 100644 --- a/htdocs/core/triggers/interface_20_all_Logevents.class.php +++ b/htdocs/core/triggers/interface_20_all_Logevents.class.php @@ -90,7 +90,7 @@ public function __construct(DoliDB $db) * @return int if KO: <0, if no trigger ran: 0, if OK: >0 * @throws Exception dol_syslog can throw Exceptions */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action, Object $object, User $user, Translate $langs, Conf $conf): int { if (getDolGlobalString('MAIN_LOGEVENTS_DISABLE_ALL')) { return 0; // Log events is disabled (hidden features) diff --git a/htdocs/core/triggers/interface_20_modWorkflow_WorkflowManager.class.php b/htdocs/core/triggers/interface_20_modWorkflow_WorkflowManager.class.php index 28204bfb5ad17..9358bb6fc5494 100644 --- a/htdocs/core/triggers/interface_20_modWorkflow_WorkflowManager.class.php +++ b/htdocs/core/triggers/interface_20_modWorkflow_WorkflowManager.class.php @@ -62,7 +62,7 @@ public function __construct($db) * @param conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,CommonObject $object, User $user, Translate $langs, Conf $conf): int { if (empty($conf->workflow) || empty($conf->workflow->enabled)) { return 0; // Module not active, we do nothing diff --git a/htdocs/core/triggers/interface_50_modAgenda_ActionsAuto.class.php b/htdocs/core/triggers/interface_50_modAgenda_ActionsAuto.class.php index 386cbb9b8ef6b..0f72cda7c3949 100644 --- a/htdocs/core/triggers/interface_50_modAgenda_ActionsAuto.class.php +++ b/htdocs/core/triggers/interface_50_modAgenda_ActionsAuto.class.php @@ -76,7 +76,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,CommonObject $object, User $user, Translate $langs, Conf $conf): int { if (!isModEnabled('agenda')) { return 0; // Module not active, we do nothing diff --git a/htdocs/core/triggers/interface_50_modBlockedlog_ActionsBlockedLog.class.php b/htdocs/core/triggers/interface_50_modBlockedlog_ActionsBlockedLog.class.php index 2f6f42f4b28da..530bf7458e3a3 100644 --- a/htdocs/core/triggers/interface_50_modBlockedlog_ActionsBlockedLog.class.php +++ b/htdocs/core/triggers/interface_50_modBlockedlog_ActionsBlockedLog.class.php @@ -56,7 +56,7 @@ public function __construct($db) * @param conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { if (!isModEnabled('blockedlog')) { return 0; // Module not active, we do nothing diff --git a/htdocs/core/triggers/interface_50_modEventOrganization_EventOrganization.class.php b/htdocs/core/triggers/interface_50_modEventOrganization_EventOrganization.class.php index f0b8b4198253e..9a78069f11d3e 100644 --- a/htdocs/core/triggers/interface_50_modEventOrganization_EventOrganization.class.php +++ b/htdocs/core/triggers/interface_50_modEventOrganization_EventOrganization.class.php @@ -69,7 +69,7 @@ public function __construct($db) * @param conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { if (empty($conf->eventorganization) || empty($conf->eventorganization->enabled)) { return 0; // Module not active, we do nothing diff --git a/htdocs/core/triggers/interface_50_modLdap_Ldapsynchro.class.php b/htdocs/core/triggers/interface_50_modLdap_Ldapsynchro.class.php index 77ad283a17843..2f6df82c2b793 100644 --- a/htdocs/core/triggers/interface_50_modLdap_Ldapsynchro.class.php +++ b/htdocs/core/triggers/interface_50_modLdap_Ldapsynchro.class.php @@ -59,7 +59,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { if (empty($conf->ldap) || empty($conf->ldap->enabled)) { return 0; // Module not active, we do nothing diff --git a/htdocs/core/triggers/interface_50_modMailmanspip_Mailmanspipsynchro.class.php b/htdocs/core/triggers/interface_50_modMailmanspip_Mailmanspipsynchro.class.php index a85060582c591..ea4f6176eb309 100644 --- a/htdocs/core/triggers/interface_50_modMailmanspip_Mailmanspipsynchro.class.php +++ b/htdocs/core/triggers/interface_50_modMailmanspip_Mailmanspipsynchro.class.php @@ -58,7 +58,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { if (empty($conf->mailmanspip) || empty($conf->mailmanspip->enabled)) { return 0; // Module not active, we do nothing diff --git a/htdocs/core/triggers/interface_50_modNotification_Notification.class.php b/htdocs/core/triggers/interface_50_modNotification_Notification.class.php index 86bf821cd365a..5ca0ff2d860c8 100644 --- a/htdocs/core/triggers/interface_50_modNotification_Notification.class.php +++ b/htdocs/core/triggers/interface_50_modNotification_Notification.class.php @@ -67,7 +67,7 @@ public function __construct($db) * @param conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { global $hookmanager; diff --git a/htdocs/core/triggers/interface_50_modTicket_TicketEmail.class.php b/htdocs/core/triggers/interface_50_modTicket_TicketEmail.class.php index 9661cdb6416a8..8cc37800da0a6 100644 --- a/htdocs/core/triggers/interface_50_modTicket_TicketEmail.class.php +++ b/htdocs/core/triggers/interface_50_modTicket_TicketEmail.class.php @@ -59,7 +59,7 @@ public function __construct($db) * @param conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { global $mysoc; diff --git a/htdocs/core/triggers/interface_80_modStripe_Stripe.class.php b/htdocs/core/triggers/interface_80_modStripe_Stripe.class.php index 89ab8a634c42c..99af8b32d9a76 100644 --- a/htdocs/core/triggers/interface_80_modStripe_Stripe.class.php +++ b/htdocs/core/triggers/interface_80_modStripe_Stripe.class.php @@ -64,7 +64,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { // Put here code you want to execute when a Dolibarr business event occurs. // Data and type of action are stored into $object and $action diff --git a/htdocs/core/triggers/interface_90_modSociete_ContactRoles.class.php b/htdocs/core/triggers/interface_90_modSociete_ContactRoles.class.php index 4c2aaf7b26a98..12944752e2083 100644 --- a/htdocs/core/triggers/interface_90_modSociete_ContactRoles.class.php +++ b/htdocs/core/triggers/interface_90_modSociete_ContactRoles.class.php @@ -65,7 +65,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,CommonObject $object, User $user, Translate $langs, Conf $conf): int { if ($action === 'PROPAL_CREATE' || $action === 'ORDER_CREATE' || $action === 'BILL_CREATE' || $action === 'ORDER_SUPPLIER_CREATE' || $action === 'BILL_SUPPLIER_CREATE' || $action === 'PROPOSAL_SUPPLIER_CREATE' diff --git a/htdocs/core/triggers/interface_95_modWebhook_WebhookTriggers.class.php b/htdocs/core/triggers/interface_95_modWebhook_WebhookTriggers.class.php index 2d16d54ed99aa..608e2cdba8598 100644 --- a/htdocs/core/triggers/interface_95_modWebhook_WebhookTriggers.class.php +++ b/htdocs/core/triggers/interface_95_modWebhook_WebhookTriggers.class.php @@ -64,7 +64,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,CommonObject $object, User $user, Translate $langs, Conf $conf): int { if (empty($conf->webhook) || empty($conf->webhook->enabled)) { return 0; // If module is not enabled, we do nothing diff --git a/htdocs/core/triggers/interface_95_modZapier_ZapierTriggers.class.php b/htdocs/core/triggers/interface_95_modZapier_ZapierTriggers.class.php index d6ed6a6383f77..f406f679c5b35 100644 --- a/htdocs/core/triggers/interface_95_modZapier_ZapierTriggers.class.php +++ b/htdocs/core/triggers/interface_95_modZapier_ZapierTriggers.class.php @@ -58,7 +58,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,Object $object, User $user, Translate $langs, Conf $conf): int { if (empty($conf->zapier) || empty($conf->zapier->enabled)) { // Module not active, we do nothing diff --git a/htdocs/knowledgemanagement/core/modules/knowledgemanagement/mod_knowledgerecord_standard.php b/htdocs/knowledgemanagement/core/modules/knowledgemanagement/mod_knowledgerecord_standard.php index d12865c47c01e..4d62d520439d4 100644 --- a/htdocs/knowledgemanagement/core/modules/knowledgemanagement/mod_knowledgerecord_standard.php +++ b/htdocs/knowledgemanagement/core/modules/knowledgemanagement/mod_knowledgerecord_standard.php @@ -83,7 +83,7 @@ public function getExample() * @param Object $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/knowledgemanagement/core/modules/knowledgemanagement/modules_knowledgerecord.php b/htdocs/knowledgemanagement/core/modules/knowledgemanagement/modules_knowledgerecord.php index 6de5bc1043153..a6b6c77a40e0f 100644 --- a/htdocs/knowledgemanagement/core/modules/knowledgemanagement/modules_knowledgerecord.php +++ b/htdocs/knowledgemanagement/core/modules/knowledgemanagement/modules_knowledgerecord.php @@ -133,7 +133,7 @@ public function getExample() * @param Object $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { return true; } diff --git a/htdocs/modulebuilder/template/class/actions_mymodule.class.php b/htdocs/modulebuilder/template/class/actions_mymodule.class.php index 62929391eba5a..1578aa962a9f3 100644 --- a/htdocs/modulebuilder/template/class/actions_mymodule.class.php +++ b/htdocs/modulebuilder/template/class/actions_mymodule.class.php @@ -68,7 +68,7 @@ class ActionsMyModule extends CommonHookActions * * @param DoliDB $db Database handler */ - public function __construct($db) + public function __construct(DoliDB $db) { $this->db = $db; } @@ -84,7 +84,7 @@ public function __construct($db) * =0 if OK but we want to process standard actions too, * >0 if OK and we want to replace standard actions. */ - public function getNomUrl($parameters, &$object, &$action) + public function getNomUrl(mixed $parameters,CommonObject &$object,string &$action): int { global $db, $langs, $conf, $user; $this->resprints = ''; @@ -100,7 +100,7 @@ public function getNomUrl($parameters, &$object, &$action) * @param HookManager $hookmanager Hook manager propagated to allow calling another hook * @return int Return integer < 0 on error, 0 on success, 1 to replace standard code */ - public function doActions($parameters, &$object, &$action, $hookmanager) + public function doActions(mixed $parameters,CommonObject &$object,?string &$action,HookManager $hookmanager): int { global $conf, $user, $langs; @@ -135,7 +135,7 @@ public function doActions($parameters, &$object, &$action, $hookmanager) * @param HookManager $hookmanager Hook manager propagated to allow calling another hook * @return int Return integer < 0 on error, 0 on success, 1 to replace standard code */ - public function doMassActions($parameters, &$object, &$action, $hookmanager) + public function doMassActions(mixed $parameters,CommonObject &$object,?string &$action,HookManager $hookmanager): int { global $conf, $user, $langs; @@ -171,7 +171,7 @@ public function doMassActions($parameters, &$object, &$action, $hookmanager) * @param HookManager $hookmanager Hook manager propagated to allow calling another hook * @return int Return integer < 0 on error, 0 on success, 1 to replace standard code */ - public function addMoreMassActions($parameters, &$object, &$action, $hookmanager) + public function addMoreMassActions(mixed $parameters,CommonObject &$object,?string &$action,HookManager $hookmanager): int { global $conf, $user, $langs; @@ -203,7 +203,7 @@ public function addMoreMassActions($parameters, &$object, &$action, $hookmanager * =0 if OK but we want to process standard actions too, * >0 if OK and we want to replace standard actions. */ - public function beforePDFCreation($parameters, &$object, &$action) + public function beforePDFCreation(mixed $parameters,CommonObject &$object,string &$action): int { global $conf, $user, $langs; global $hookmanager; @@ -232,7 +232,7 @@ public function beforePDFCreation($parameters, &$object, &$action) * =0 if OK but we want to process standard actions too, * >0 if OK and we want to replace standard actions. */ - public function afterPDFCreation($parameters, &$pdfhandler, &$action) + public function afterPDFCreation(mixed $parameters,CommonDocGenerator &$pdfhandler,string &$action): int { global $conf, $user, $langs; global $hookmanager; @@ -262,7 +262,7 @@ public function afterPDFCreation($parameters, &$pdfhandler, &$action) * @param HookManager $hookmanager Hook manager propagated to allow calling another hook * @return int Return integer < 0 on error, 0 on success, 1 to replace standard code */ - public function loadDataForCustomReports($parameters, &$action, $hookmanager) + public function loadDataForCustomReports(mixed $parameters,?string &$action,HookManager $hookmanager): int { global $langs; @@ -309,7 +309,7 @@ public function loadDataForCustomReports($parameters, &$action, $hookmanager) * =0 if OK but we want to process standard actions too, * >0 if OK and we want to replace standard actions. */ - public function restrictedArea($parameters, &$action, $hookmanager) + public function restrictedArea(mixed $parameters, string &$action,HookManager $hookmanager): int { global $user; @@ -337,7 +337,7 @@ public function restrictedArea($parameters, &$action, $hookmanager) * =0 if OK but we want to process standard actions too, * >0 if OK and we want to replace standard actions. */ - public function completeTabsHead(&$parameters, &$object, &$action, $hookmanager) + public function completeTabsHead(mixed &$parameters,CommonObject &$object,string &$action,HookManager $hookmanager): int { global $langs, $conf, $user; diff --git a/htdocs/modulebuilder/template/class/api_mymodule.class.php b/htdocs/modulebuilder/template/class/api_mymodule.class.php index 14b89da504c7b..2c3df0534f739 100644 --- a/htdocs/modulebuilder/template/class/api_mymodule.class.php +++ b/htdocs/modulebuilder/template/class/api_mymodule.class.php @@ -75,7 +75,7 @@ public function __construct() * @throws RestException 403 Not allowed * @throws RestException 404 Not found */ - public function get($id) + public function get(int $id) { if (!DolibarrApiAccess::$user->hasRight('mymodule', 'myobject', 'read')) { throw new RestException(403); @@ -113,7 +113,7 @@ public function get($id) * * @url GET /myobjects/ */ - public function index($sortfield = "t.rowid", $sortorder = 'ASC', $limit = 100, $page = 0, $sqlfilters = '', $properties = '') + public function index(string $sortfield = "t.rowid",string $sortorder = 'ASC',int $limit = 100,int $page = 0,string $sqlfilters = '',string $properties = ''): array { $obj_ret = array(); $tmpobject = new MyObject($this->db); @@ -203,7 +203,7 @@ public function index($sortfield = "t.rowid", $sortorder = 'ASC', $limit = 100, * * @url POST myobjects/ */ - public function post($request_data = null) + public function post(array $request_data = null): int { if (!DolibarrApiAccess::$user->hasRight('mymodule', 'myobject', 'write')) { throw new RestException(403); @@ -257,7 +257,7 @@ public function post($request_data = null) * * @url PUT myobjects/{id} */ - public function put($id, $request_data = null) + public function put(int $id,array $request_data = null) { if (!DolibarrApiAccess::$user->hasRight('mymodule', 'myobject', 'write')) { throw new RestException(403); @@ -316,7 +316,7 @@ public function put($id, $request_data = null) * * @url DELETE myobjects/{id} */ - public function delete($id) + public function delete(int $id): array { if (!DolibarrApiAccess::$user->hasRight('mymodule', 'myobject', 'delete')) { throw new RestException(403); @@ -357,7 +357,7 @@ public function delete($id) * * @throws RestException */ - private function _validateMyObject($data) + private function _validateMyObject(array $data): array { $myobject = array(); foreach ($this->myobject->fields as $field => $propfield) { diff --git a/htdocs/modulebuilder/template/class/myobject.class.php b/htdocs/modulebuilder/template/class/myobject.class.php index f9251809d17bb..fe6ee71d7ff09 100644 --- a/htdocs/modulebuilder/template/class/myobject.class.php +++ b/htdocs/modulebuilder/template/class/myobject.class.php @@ -280,7 +280,7 @@ public function __construct(DoliDB $db) * @param int<0,1> $notrigger 0=launch triggers after, 1=disable triggers * @return int<-1,max> Return integer <0 if KO, Id of created object if OK */ - public function create(User $user, $notrigger = 0) + public function create(User $user, int $notrigger = 0): int { $resultcreate = $this->createCommon($user, $notrigger); @@ -298,7 +298,7 @@ public function create(User $user, $notrigger = 0) * @param int $fromid Id of object to clone * @return self|int<-1,-1> New object created, <0 if KO */ - public function createFromClone(User $user, $fromid) + public function createFromClone(User $user, int $fromid) { global $langs, $extrafields; $error = 0; @@ -399,7 +399,7 @@ public function createFromClone(User $user, $fromid) * @param int<0,1> $nolines 0=Default to load extrafields, 1=No extrafields * @return int<-1,1> Return integer <0 if KO, 0 if not found, >0 if OK */ - public function fetch($id, $ref = null, $noextrafields = 0, $nolines = 0) + public function fetch(int $id, string $ref = null, int $noextrafields = 0, int $nolines = 0): int { $result = $this->fetchCommon($id, $ref, '', $noextrafields); if ($result > 0 && !empty($this->table_element_line) && empty($nolines)) { @@ -414,7 +414,7 @@ public function fetch($id, $ref = null, $noextrafields = 0, $nolines = 0) * @param int<0,1> $noextrafields 0=Default to load extrafields, 1=No extrafields * @return int<-1,1> Return integer <0 if KO, 0 if not found, >0 if OK */ - public function fetchLines($noextrafields = 0) + public function fetchLines(int $noextrafields = 0): int { $this->lines = array(); @@ -437,7 +437,7 @@ public function fetchLines($noextrafields = 0) * @param string $filtermode No longer used * @return array|int<-1,-1> <0 if KO, array of pages if OK */ - public function fetchAll($sortorder = '', $sortfield = '', $limit = 1000, $offset = 0, string $filter = '', $filtermode = 'AND') + public function fetchAll(string $sortorder = '',string $sortfield = '',int $limit = 1000,int $offset = 0, string $filter = '',string $filtermode = 'AND') { dol_syslog(__METHOD__, LOG_DEBUG); @@ -507,7 +507,7 @@ public function fetchAll($sortorder = '', $sortfield = '', $limit = 1000, $offse * @param int<0,1> $notrigger 0=launch triggers after, 1=disable triggers * @return int<-1,1> Return integer <0 if KO, >0 if OK */ - public function update(User $user, $notrigger = 0) + public function update(User $user,int $notrigger = 0): int { return $this->updateCommon($user, $notrigger); } @@ -519,7 +519,7 @@ public function update(User $user, $notrigger = 0) * @param int<0,1> $notrigger 0=launch triggers, 1=disable triggers * @return int<-1,1> Return integer <0 if KO, >0 if OK */ - public function delete(User $user, $notrigger = 0) + public function delete(User $user,int $notrigger = 0): int { return $this->deleteCommon($user, $notrigger); //return $this->deleteCommon($user, $notrigger, 1); @@ -533,7 +533,7 @@ public function delete(User $user, $notrigger = 0) * @param int<0,1> $notrigger 0=launch triggers after, 1=disable triggers * @return int<-2,1> >0 if OK, <0 if KO */ - public function deleteLine(User $user, $idline, $notrigger = 0) + public function deleteLine(User $user,int $idline,int $notrigger = 0): int { if ($this->status < 0) { $this->error = 'ErrorDeleteLineNotAllowedByObjectStatus'; @@ -551,7 +551,7 @@ public function deleteLine(User $user, $idline, $notrigger = 0) * @param int<0,1> $notrigger 1=Does not execute triggers, 0= execute triggers * @return int<-1,1> Return integer <=0 if OK, 0=Nothing done, >0 if KO */ - public function validate($user, $notrigger = 0) + public function validate(User $user,int $notrigger = 0): int { global $conf; @@ -687,7 +687,7 @@ public function validate($user, $notrigger = 0) * @param int<0,1> $notrigger 1=Does not execute triggers, 0=Execute triggers * @return int<0,1> Return integer <0 if KO, >0 if OK */ - public function setDraft($user, $notrigger = 0) + public function setDraft(User $user,int $notrigger = 0): int { // Protection if ($this->status <= self::STATUS_DRAFT) { @@ -711,7 +711,7 @@ public function setDraft($user, $notrigger = 0) * @param int<0,1> $notrigger 1=Does not execute triggers, 0=Execute triggers * @return int<-1,1> Return integer <0 if KO, 0=Nothing done, >0 if OK */ - public function cancel($user, $notrigger = 0) + public function cancel(User $user,int $notrigger = 0): int { // Protection if ($this->status != self::STATUS_VALIDATED) { @@ -735,7 +735,7 @@ public function cancel($user, $notrigger = 0) * @param int<0,1> $notrigger 1=Does not execute triggers, 0=Execute triggers * @return int<-1,1> Return integer <0 if KO, 0=Nothing done, >0 if OK */ - public function reopen($user, $notrigger = 0) + public function reopen(User $user,int $notrigger = 0): int { // Protection if ($this->status == self::STATUS_VALIDATED) { @@ -759,7 +759,7 @@ public function reopen($user, $notrigger = 0) * @since v18 * @return array{optimize?:string,picto?:string,ref?:string} */ - public function getTooltipContentArray($params) + public function getTooltipContentArray(mixed $params): array { global $langs; @@ -792,7 +792,7 @@ public function getTooltipContentArray($params) * @param int $save_lastsearch_value -1=Auto, 0=No save of lastsearch_values when clicking, 1=Save lastsearch_values whenclicking * @return string String with URL */ - public function getNomUrl($withpicto = 0, $option = '', $notooltip = 0, $morecss = '', $save_lastsearch_value = -1) + public function getNomUrl(int $withpicto = 0,string $option = '',int $notooltip = 0,string $morecss = '',int $save_lastsearch_value = -1): string { global $conf, $langs, $hookmanager; @@ -911,7 +911,7 @@ public function getNomUrl($withpicto = 0, $option = '', $notooltip = 0, $morecss * @param ?array $arraydata Array of data * @return string HTML Code for Kanban thumb. */ - public function getKanbanView($option = '', $arraydata = null) + public function getKanbanView(string $option = '',mixed $arraydata = null): string { global $conf, $langs; @@ -953,7 +953,7 @@ public function getKanbanView($option = '', $arraydata = null) * @param int<0,6> $mode 0=long label, 1=short label, 2=Picto + short label, 3=Picto, 4=Picto + long label, 5=Short label + Picto, 6=Long label + Picto * @return string Label of status */ - public function getLabelStatus($mode = 0) + public function getLabelStatus(int $mode = 0): string { return $this->LibStatut($this->status, $mode); } @@ -964,7 +964,7 @@ public function getLabelStatus($mode = 0) * @param int<0,6> $mode 0=long label, 1=short label, 2=Picto + short label, 3=Picto, 4=Picto + long label, 5=Short label + Picto, 6=Long label + Picto * @return string Label of status */ - public function getLibStatut($mode = 0) + public function getLibStatut(int $mode = 0): string { return $this->LibStatut($this->status, $mode); } @@ -977,7 +977,7 @@ public function getLibStatut($mode = 0) * @param int<0,6> $mode 0=long label, 1=short label, 2=Picto + short label, 3=Picto, 4=Picto + long label, 5=Short label + Picto, 6=Long label + Picto * @return string Label of status */ - public function LibStatut($status, $mode = 0) + public function LibStatut(int $status,int $mode = 0): string { // phpcs:enable if (is_null($status)) { @@ -1010,7 +1010,7 @@ public function LibStatut($status, $mode = 0) * @param int $id Id of object * @return void */ - public function info($id) + public function info(int $id): void { $sql = "SELECT rowid,"; $sql .= " date_creation as datec, tms as datem"; @@ -1064,7 +1064,7 @@ public function info($id) * * @return int */ - public function initAsSpecimen() + public function initAsSpecimen(): int { // Set here init that are not commonf fields // $this->property1 = ... @@ -1099,7 +1099,7 @@ public function getLinesArray() * * @return string Object free reference */ - public function getNextNumRef() + public function getNextNumRef(): string { global $langs, $conf; $langs->load("mymodule@mymodule"); @@ -1161,7 +1161,7 @@ public function getNextNumRef() * @param ?array $moreparams Array to provide more information * @return int 0 if KO, 1 if OK */ - public function generateDocument($modele, $outputlangs, $hidedetails = 0, $hidedesc = 0, $hideref = 0, $moreparams = null) + public function generateDocument(string $modele,Translate $outputlangs,int $hidedetails = 0,int $hidedesc = 0,int $hideref = 0,mixed $moreparams = null): int { global $langs; @@ -1198,7 +1198,7 @@ public function generateDocument($modele, $outputlangs, $hidedetails = 0, $hided * @param string $fieldValue value of attribute * @return bool Return false if fail, true on success, set $this->error for error message */ - public function validateField($fields, $fieldKey, $fieldValue) + public function validateField(mixed $fields,mixed $fieldKey,mixed $fieldValue): bool { // Add your own validation rules here. // ... @@ -1213,7 +1213,7 @@ public function validateField($fields, $fieldKey, $fieldValue) * * @return int 0 if OK, <>0 if KO (this function is used also by cron so only 0 is OK) */ - public function doScheduledJob() + public function doScheduledJob(): int { //global $conf, $langs; diff --git a/htdocs/modulebuilder/template/core/boxes/mymodulewidget1.php b/htdocs/modulebuilder/template/core/boxes/mymodulewidget1.php index 9660c33544d98..44787528a3365 100644 --- a/htdocs/modulebuilder/template/core/boxes/mymodulewidget1.php +++ b/htdocs/modulebuilder/template/core/boxes/mymodulewidget1.php @@ -74,7 +74,7 @@ class mymodulewidget1 extends ModeleBoxes * @param DoliDB $db Database handler * @param string $param More parameters */ - public function __construct(DoliDB $db, $param = '') + public function __construct(DoliDB $db,string $param = '') { global $user; @@ -94,7 +94,7 @@ public function __construct(DoliDB $db, $param = '') * @param int<0,max> $max Maximum number of records to load * @return void */ - public function loadBox($max = 5) + public function loadBox(int $max = 5): void { global $langs; @@ -196,7 +196,7 @@ public function loadBox($max = 5) * @param int<0,1> $nooutput No print, only return string * @return string */ - public function showBox($head = null, $contents = null, $nooutput = 0) + public function showBox(?array $head = null,?array $contents = null,int $nooutput = 0): string { // You may make your own code here… // … or use the parent's class function using the provided head and contents templates diff --git a/htdocs/modulebuilder/template/core/modules/mailings/mailing_mymodule_selector1.modules.php b/htdocs/modulebuilder/template/core/modules/mailings/mailing_mymodule_selector1.modules.php index a14f339306b72..eaad3fd30ca27 100644 --- a/htdocs/modulebuilder/template/core/modules/mailings/mailing_mymodule_selector1.modules.php +++ b/htdocs/modulebuilder/template/core/modules/mailings/mailing_mymodule_selector1.modules.php @@ -52,7 +52,7 @@ class mailing_mailing_mymodule_selector1 extends MailingTargets * * @param DoliDB $db Database handler */ - public function __construct($db) + public function __construct(DoliDB $db) { $this->db = $db; //$this->enabled = ... @@ -64,7 +64,7 @@ public function __construct($db) * * @return string Return select zone */ - public function formFilter() + public function formFilter(): string { global $langs; $langs->load("members"); @@ -91,7 +91,7 @@ public function formFilter() * @param int $id ID * @return string Url link */ - public function url($id) + public function url(int $id): string { return ''.img_object('', "generic").''; } @@ -104,7 +104,7 @@ public function url($id) * @param int $mailing_id Id of emailing * @return int<-1,max> Return integer <0 if error, number of emails added if ok */ - public function add_to_target($mailing_id) + public function add_to_target(int $mailing_id): int { // phpcs:enable $target = array(); @@ -173,7 +173,7 @@ public function add_to_target($mailing_id) * * @return string[] Array with SQL requests */ - public function getSqlArrayForStats() + public function getSqlArrayForStats(): array { // CHANGE THIS: Optional @@ -192,7 +192,7 @@ public function getSqlArrayForStats() * @param string $sql Not used here * @return int<-1,max> Nb of recipients or -1 if KO */ - public function getNbOfRecipients($sql = '') + public function getNbOfRecipients(string $sql = ''): int { $sql = "SELECT COUNT(DISTINCT(email)) as nb"; $sql .= " FROM ".MAIN_DB_PREFIX."myobject as p"; diff --git a/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_advanced.php b/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_advanced.php index c8117863f12e4..027da9ac6c7d3 100644 --- a/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_advanced.php +++ b/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_advanced.php @@ -59,7 +59,7 @@ class mod_myobject_advanced extends ModeleNumRefMyObject * @param Translate $langs Translate Object * @return string Text with description */ - public function info($langs) + public function info(Translate $langs): string { global $db; @@ -98,7 +98,7 @@ public function info($langs) * * @return string Example */ - public function getExample() + public function getExample(): string { global $db, $langs; @@ -127,7 +127,7 @@ public function getExample() * @param MyObject $object Object we need next value for * @return string|int<-1,0> Next value if OK, <=0 if KO */ - public function getNextValue($object) + public function getNextValue(MyObject $object) { global $db; diff --git a/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_standard.php b/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_standard.php index 4d2329d8a2f85..f4381b30b9127 100644 --- a/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_standard.php +++ b/htdocs/modulebuilder/template/core/modules/mymodule/mod_myobject_standard.php @@ -60,7 +60,7 @@ class mod_myobject_standard extends ModeleNumRefMyObject * @param Translate $langs Translate Object * @return string Text with description */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("SimpleNumRefModelDesc", $this->prefix); } @@ -71,7 +71,7 @@ public function info($langs) * * @return string Example */ - public function getExample() + public function getExample(): string { return $this->prefix."0501-0001"; } @@ -84,7 +84,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return bool false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(CommonObject $object): bool { global $conf, $langs, $db; @@ -124,7 +124,7 @@ public function canBeActivated($object) * @param MyObject $object Object we need next value for * @return string|int<-1,0> Next value if OK, <=0 if KO */ - public function getNextValue($object) + public function getNextValue(MyObject $object) { global $db, $conf; diff --git a/htdocs/modulebuilder/template/core/modules/mymodule/modules_myobject.php b/htdocs/modulebuilder/template/core/modules/mymodule/modules_myobject.php index ea6858106e957..2ebe71cf3b37e 100644 --- a/htdocs/modulebuilder/template/core/modules/mymodule/modules_myobject.php +++ b/htdocs/modulebuilder/template/core/modules/mymodule/modules_myobject.php @@ -47,7 +47,7 @@ abstract class ModelePDFMyObject extends CommonDocGenerator * @param int<0,max> $maxfilenamelength Max length of value to show * @return string[]|int<-1,0> List of templates */ - public static function liste_modeles($db, $maxfilenamelength = 0) + public static function liste_modeles(DoliDB $db,int $maxfilenamelength = 0): array { // phpcs:enable $type = 'myobject'; @@ -71,7 +71,7 @@ public static function liste_modeles($db, $maxfilenamelength = 0) * @param int<0,1> $hideref Do not show ref * @return int<-1,1> 1 if OK, <=0 if KO */ - abstract public function write_file($object, $outputlangs, $srctemplatepath = '', $hidedetails = 0, $hidedesc = 0, $hideref = 0); + abstract public function write_file(MyObject $object,Translate $outputlangs,string $srctemplatepath = '',int $hidedetails = 0,int $hidedesc = 0,int $hideref = 0): int; } @@ -85,7 +85,7 @@ abstract class ModeleNumRefMyObject extends CommonNumRefGenerator * * @return string Example */ - abstract public function getExample(); + abstract public function getExample(): string; /** * Return next free value @@ -93,5 +93,5 @@ abstract public function getExample(); * @param MyObject $object Object we need next value for * @return string|int<-1,0> Next value if OK, <=0 if KO */ - abstract public function getNextValue($object); + abstract public function getNextValue(MyObject $object); } diff --git a/htdocs/modulebuilder/template/core/triggers/interface_99_modMyModule_MyModuleTriggers.class.php b/htdocs/modulebuilder/template/core/triggers/interface_99_modMyModule_MyModuleTriggers.class.php index b9391aa6afacb..38cb8b935a3c4 100644 --- a/htdocs/modulebuilder/template/core/triggers/interface_99_modMyModule_MyModuleTriggers.class.php +++ b/htdocs/modulebuilder/template/core/triggers/interface_99_modMyModule_MyModuleTriggers.class.php @@ -44,7 +44,7 @@ class InterfaceMyModuleTriggers extends DolibarrTriggers * * @param DoliDB $db Database handler */ - public function __construct($db) + public function __construct(DoliDB $db) { parent::__construct($db); $this->family = "demo"; @@ -65,7 +65,7 @@ public function __construct($db) * @param Conf $conf Object conf * @return int Return integer <0 if KO, 0 if no triggered ran, >0 if OK */ - public function runTrigger($action, $object, User $user, Translate $langs, Conf $conf) + public function runTrigger(string $action,CommonObject $object, User $user, Translate $langs, Conf $conf): int { if (!isModEnabled('mymodule')) { return 0; // If module is not enabled, we do nothing diff --git a/htdocs/modulebuilder/template/lib/mymodule.lib.php b/htdocs/modulebuilder/template/lib/mymodule.lib.php index d432a35b4afea..42df87a7a774c 100644 --- a/htdocs/modulebuilder/template/lib/mymodule.lib.php +++ b/htdocs/modulebuilder/template/lib/mymodule.lib.php @@ -26,7 +26,7 @@ * * @return array */ -function mymoduleAdminPrepareHead() +function mymoduleAdminPrepareHead(): array { global $langs, $conf; diff --git a/htdocs/modulebuilder/template/lib/mymodule_myobject.lib.php b/htdocs/modulebuilder/template/lib/mymodule_myobject.lib.php index e63dc663ae5d6..1a8cf3cd827ab 100644 --- a/htdocs/modulebuilder/template/lib/mymodule_myobject.lib.php +++ b/htdocs/modulebuilder/template/lib/mymodule_myobject.lib.php @@ -27,7 +27,7 @@ * @param MyObject $object MyObject * @return array Array of tabs */ -function myobjectPrepareHead($object) +function myobjectPrepareHead($object): array { global $db, $langs, $conf; diff --git a/htdocs/partnership/core/modules/partnership/mod_partnership_advanced.php b/htdocs/partnership/core/modules/partnership/mod_partnership_advanced.php index 1c677e2b28273..57dea9dd7b113 100644 --- a/htdocs/partnership/core/modules/partnership/mod_partnership_advanced.php +++ b/htdocs/partnership/core/modules/partnership/mod_partnership_advanced.php @@ -58,7 +58,7 @@ class mod_partnership_advanced extends ModeleNumRefPartnership * @param Translate $langs Lang object to use for output * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db; diff --git a/htdocs/partnership/core/modules/partnership/mod_partnership_standard.php b/htdocs/partnership/core/modules/partnership/mod_partnership_standard.php index 90c84a2909f55..c4c6040c84c9c 100644 --- a/htdocs/partnership/core/modules/partnership/mod_partnership_standard.php +++ b/htdocs/partnership/core/modules/partnership/mod_partnership_standard.php @@ -60,7 +60,7 @@ class mod_partnership_standard extends ModeleNumRefPartnership * @param Translate $langs Lang object to use for output * @return string Text with description */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("SimpleNumRefModelDesc", $this->prefix); } @@ -84,7 +84,7 @@ public function getExample() * @param Object $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_advanced.php b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_advanced.php index 5be97a4598867..1ad405aa4fa7c 100644 --- a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_advanced.php +++ b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_advanced.php @@ -58,7 +58,7 @@ class mod_recruitmentcandidature_advanced extends ModeleNumRefRecruitmentCandida * @param Translate $langs Language * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db; diff --git a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_standard.php b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_standard.php index 8a30e86e0f24a..8997c45ce4f9a 100644 --- a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_standard.php +++ b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentcandidature_standard.php @@ -60,7 +60,7 @@ class mod_recruitmentcandidature_standard extends ModeleNumRefRecruitmentCandida * @param Translate $langs Language * @return string Text with description */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("SimpleNumRefModelDesc", $this->prefix); } @@ -84,7 +84,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db; diff --git a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_advanced.php b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_advanced.php index 0ada1a5e802c4..ad49117c2fa20 100644 --- a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_advanced.php +++ b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_advanced.php @@ -58,7 +58,7 @@ class mod_recruitmentjobposition_advanced extends ModeleNumRefRecruitmentJobPosi * @param Translate $langs Language * @return string Descriptive text */ - public function info($langs) + public function info(Translate $langs): string { global $db; diff --git a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_standard.php b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_standard.php index fdfe40d948c74..b2b9111969491 100644 --- a/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_standard.php +++ b/htdocs/recruitment/core/modules/recruitment/mod_recruitmentjobposition_standard.php @@ -60,7 +60,7 @@ class mod_recruitmentjobposition_standard extends ModeleNumRefRecruitmentJobPosi * @param Translate $langs Language * @return string Text with description */ - public function info($langs) + public function info(Translate $langs): string { return $langs->trans("SimpleNumRefModelDesc", $this->prefix); } @@ -84,7 +84,7 @@ public function getExample() * @param CommonObject $object Object we need next value for * @return boolean false if conflict, true if ok */ - public function canBeActivated($object) + public function canBeActivated(Object $object): bool { global $conf, $langs, $db;