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

FeedInput is using deprecated type definition #4047

Open
pixel-fabian opened this issue May 13, 2024 · 2 comments
Open

FeedInput is using deprecated type definition #4047

pixel-fabian opened this issue May 13, 2024 · 2 comments

Comments

@pixel-fabian
Copy link

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
Copy link
Member

i556484 commented May 13, 2024

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
Copy link

Hi @pixel-fabian ,

We are discussing this internally and provide an update soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants