Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[Dependency] Missing InfluxDB dependency in cmd.class.php #3002

Closed
kwizer15 opened this issue Dec 20, 2024 · 1 comment
Closed

[Dependency] Missing InfluxDB dependency in cmd.class.php #3002

kwizer15 opened this issue Dec 20, 2024 · 1 comment

Comments

@kwizer15
Copy link
Contributor

kwizer15 commented Dec 20, 2024

Description

The cmd.class.php file uses InfluxDB functionalities but the dependency is not declared in the project.

Impact

This can cause runtime errors when trying to use InfluxDB features as the required classes and methods may not be available.

Current State

The following methods use InfluxDB but may fail:

  • computeInfluxData()
  • getInflux()
  • pushInflux()
  • dropInfluxDatabase()
  • dropInflux()
  • historyInfluxAll()
  • sendHistoryInflux()

Suggested Solution

Add InfluxDB PHP client as a dependency in composer.json:

{
    "require": {
        "influxdata/influxdb-php": "^3.6" # last version
    }
}

Additional Notes

@zoic21
Copy link
Contributor

zoic21 commented Jan 5, 2025

Fix with 77d1f7f

@zoic21 zoic21 closed this as completed Jan 5, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants