1328 - Code refactoring to ensure consistent variable naming

[legendword]

Changes

• Use camelCase for the names of all local variables and arguments in the backend PHP code.
• Use camelCase for all property names in the wpData object that is injected to the frontend.
• Rename "Kaltura Unique Configuration" to "Kaltura UI Configuration". The reason for this change is that the field is actually UI configuration and was misread as unique configuration when we first implemented the features. For backwards compatibility, the uniqueConfiguration property in the typeData of a node is still checked so that old Kaltura nodes continue to function.

  Issue Linkage

  Closes #1328

  PR Dependency

  Depends on: N/A

  Automated Testing

• N/A

[cypress[bot]]

4 failed and 1 flaky tests on run #4066 ↗︎

4

96

0

0

1

Details:

Fix use of old vue_uri variable name
Project: tapestry-wp	Commit: bf63710a6c
Status: Failed	Duration: 04:44 💡
Started: Sep 17, 2023 7:11 PM	Ended: Sep 17, 2023 7:16 PM

  cypress/integration/lightbox/video.spec.js • 4 failed tests

View Output Video

Test		Artifacts
Video > should be able to add a video node via Kaltura and use regular player		Output Screenshots Video
Video > should be able to add a video node via Kaltura and use Kaltura player		Output Screenshots Video
Video > adding a kaltura video should also set a thumbnail		Output Screenshots Video
Video > should be able to add a caption to a video		Output Screenshots Video

  cypress/integration/lightbox/multicontent.spec.js • 1 flaky test

View Output Video

Test		Artifacts
Multi-content > In-place > should be able to add/edit/delete multi-content child node in-place		Output Screenshots Video

_{This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.}

[legendword]

Need to update the name of the CI secret for "KALTURA_UNIQUE_CONFIG" to "KALTURA_UI_CONFIG" on GitHub in order for the Cypress tests to pass.