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

Improve Data Input Methods editability and message handling #1184

Closed
netniV opened this issue Jan 7, 2018 · 5 comments
Closed

Improve Data Input Methods editability and message handling #1184

netniV opened this issue Jan 7, 2018 · 5 comments
Labels
enhancement General tag for an enhancement

Comments

@netniV
Copy link
Member

netniV commented Jan 7, 2018

I updated a data input template with a new field and updated the associated php script to expect the new field. However, the field did not come into play until I went back and saved the data source. If there had been a lot of data sources associated with the template, this would have taken some time.

Is there an easy way to update these? Shouldn't saving the template provide

  • a prompt that to update sources that are based upon it
  • a button to update the data sources that depend on the tempalte when editing a data input
  • an option to reapply the data input change from the selection/search list if a data input is selected
@netniV netniV changed the title Updating a data_source template's input string does not update data_sources Updating a data_input template's input string does not update data_sources Jan 7, 2018
@cigamit
Copy link
Member

cigamit commented Jan 12, 2018

There should be a push_out_data_input_method() function in the lib/utility.php. Maybe it has an issue with not repopulating the poller cache for items that are impacted by the change. It's entirely possible that was a late code change in the 0.8.8 branch.

@cigamit cigamit added the bug Undesired behaviour label Jan 12, 2018
@cigamit
Copy link
Member

cigamit commented Feb 25, 2018

I did a cursory review of what is happening in data_templates.php and I believe that the data_templates.php save process is a bit messy to say the least. Though it has kind of worked for a long time, there is a lot of wasted cycles spent on things that don't have to be spent on. Plus there are some dangerous transactions.

It's all a little too much work for 1.1.36. But it seems we can resolve this issue, and increase performance when updating Data Templates at the same time.

@netniV
Copy link
Member Author

netniV commented Aug 2, 2018

I wonder if any of our recent changes have addressed this or if it's still an ongoing issue?

@cigamit
Copy link
Member

cigamit commented Aug 7, 2018

No changes as of now. I'm improving the messages for the time being, and then we can open a new issue. The problem with blindly pushing out is going to be with the law of unintended consequences.

@cigamit cigamit changed the title Updating a data_input template's input string does not update data_sources Prevent users from breaking data input methods after Data Templates or Data Sources are created Aug 7, 2018
@cigamit cigamit added enhancement General tag for an enhancement and removed bug Undesired behaviour labels Aug 7, 2018
@cigamit cigamit changed the title Prevent users from breaking data input methods after Data Templates or Data Sources are created Improve Data Input Methods editability and message handling Aug 7, 2018
cigamit added a commit that referenced this issue Aug 7, 2018
Improve Data Input Methods editability and message handling
@cigamit
Copy link
Member

cigamit commented Aug 7, 2018

New messages and removal of editing of output fields added, so I'll close this for now.

@cigamit cigamit closed this as completed Aug 7, 2018
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 30, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement General tag for an enhancement
Projects
None yet
Development

No branches or pull requests

2 participants