Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in / Register
  • B bitrix
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Package Registry
    • Container Registry
    • Infrastructure Registry
  • Monitor
    • Monitor
    • Metrics
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Yuri Yurev
  • bitrix
  • Merge requests
  • !546

Refactor code for client ID validation and syntax consistency

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Yuri Yurev requested to merge dev-1401942 into master Feb 06, 2024
  • Overview 0
  • Commits 2
  • Pipelines 1
  • Changes 1

The commit updates Legacy/Tasks.php and Legacy/Client.php to ensure valid client ID executions and improve syntax consistency. Fixed the call for setOntraportTagById method by checking if client ID is greater than zero and restructured the code to enforce consistent spacing and parentheses usage across different modules. This improves code readability and ensures more robust error checking ultimately reducing potential runtime errors.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: dev-1401942