SAP / openui5

OpenUI5 lets you build enterprise-ready web applications, responsive to all devices, running on almost any browser of your choice.
http://openui5.org
Apache License 2.0
2.9k stars 1.23k forks source link

FeedInput is using deprecated type definition #4047

Open pixel-fabian opened 1 month ago

pixel-fabian commented 1 month ago

OpenUI5 version: 1.120.12

Browser/version (+device/version): Chrome

Any other tested browsers/devices(OK/FAIL): Firefox (FAIL)

URL (minimal example if possible): https://openui5.hana.ondemand.com/entity/sap.m.FeedInput/sample/sap.m.sample.FeedInput

Steps to reproduce the problem:

  1. Open a view where the sap.m.FeedInput is used

What is the expected result? No console error

What happens instead? Console error: [DEPRECATED] The type 'sap.ui.core.TooltipBase' was accessed via globals. Defining types via globals is deprecated. In case the referenced type is an enum: require the module 'sap/ui/base/DataType' and call the static 'DataType.registerEnum' API. In case the referenced type is non-primitive, please note that only primitive types (and those derived from them) are supported for ManagedObject properties.

General annotaion: I'm wondering why deprecation warnings are presented as errors. This introduced a lot more efford than we expected on our side on a bugfix LTS update (1.120.0 to 1.120.12). We went through our whole codebase and fixed all type declerations in our source code, which is not what i expect from a bugfix release. IMHO these should have been presented as warnings to fix until the next major or minor release.

i556484 commented 1 month ago

Hello @pixel-fabian, Thank you for addressing the issue in the control code. I've created an internal incident DINC0149840. The status of the issue will be updated here in GitHub.

Abhi1897 commented 1 month ago

Hi @pixel-fabian ,

We are discussing this internally and provide an update soon.

Abhi1897 commented 4 days ago

Hi @pixel-fabian ,

We acknowledge the issue reported and we have created a backlog item (FIORITECHP1-31086) for the same. The issue will be addressed.