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

[OpenUI5Support]: Version 1.120 Fatal Issue due to Formatting function being renamed #7871

Closed
1 task done
i832513 opened this issue Nov 16, 2023 · 8 comments · Fixed by #7872
Closed
1 task done
Labels
bug This issue is a bug in the code High Prio SAP SF TOPIC Core
Milestone

Comments

@i832513
Copy link

i832513 commented Nov 16, 2023

Bug Description

Please refer to the following links:

  1. https://jira.tools.sap/browse/[CPOUI5MODELS-1487](https://jira.tools.sap/browse/CPOUI5MODELS-1487)
  2. SAP/openui5@cccc043

The function Formatting.setLegacyDateCalendarCustomizing is renamed in version 1.120 - it is unclear at this time if OpenUI5 will deliver a new version of OpenUI5 which provides the backwards compatibility necessary to prevent fatal page errors.

Affected Component

OpenUI5Support

Expected Behaviour

The OpenUI5Support static import should not crash the page no matter what version of OpenUI5 is being used.

Isolated Example

https://codesandbox.io/s/ui5-webcomponents-forked-5kchxn?file=/index.html

Steps to Reproduce

  1. Include OpenUI5 1.120.0
  2. Include OpenUI5Support feature statically

This will break the page and not load any web components.

Log Output, Stack Trace or Screenshots

OpenUI5Support.ts:140 Uncaught TypeError: i.getLegacyDateCalendarCustomizing is not a function
at Function.value (OpenUI5Support.ts:140:48)
at InitialConfiguration.ts:187:39
at w (InitialConfiguration.ts:203:2)
at l (InitialConfiguration.ts:46:2)
at l (Theme.ts:16:14)
at d (Theme.ts:71:23)
at 292 (nav-back.js:5:18)
at webpack_require (bootstrap:19:22)
at fetchLocalizedContent.js:173:1
at index.tsx:29:3

Priority

High

UI5 Web Components Version

1.19

Browser

Chrome, Edge, Firefox, Safari

Operating System

Any

Additional Context

Compatibility with version 1.120.0

Organization

SuccessFactors

Declaration

  • I’m not disclosing any internal or sensitive information.
@i832513 i832513 added the bug This issue is a bug in the code label Nov 16, 2023
@i832513
Copy link
Author

i832513 commented Nov 16, 2023

I'm having trouble right now to get the sandbox to reproduce the issue because openui5 script is not downloading or executing - I'm not sure what i did wrong - but someone could fix that.

https://codesandbox.io/s/ui5-webcomponents-forked-5kchxn?file=/index.html

@pskelin
Copy link
Contributor

pskelin commented Nov 17, 2023

It seems that your sample works with 1.120.0, but if you change the bootstrap to nightly like here, it does break.
https://codesandbox.io/s/ui5-webcomponents-forked-hj8k3z?file=/index.html:1102-1167

Working on a fix right now.

@ilhan007
Copy link
Member

Hi @scameron we plan to downport this fix for 1.19 as 1.19.1

Do you need this for older versions?

@scameron
Copy link
Contributor

I don't think we're waiting for this fix on our side. Did you mean to tag @i832513 instead?

@ilhan007
Copy link
Member

Hello Scott @i832513 we plan to downport this fix for 1.19 as 1.19.1

Do you need this for older versions?

@i832513
Copy link
Author

i832513 commented Nov 17, 2023

@ilhan007 ,
I know that the project having this issue is currently on 1.15.x - but I don't know how difficult it would be to upgrade that project to version 1.19.1 as part of the patch, if there are no major issues with that we can consider to upgrade to 1.19.1 to fix the issue.

@ilhan007
Copy link
Member

Ok, @i832513 let me know if you would be able to upgrade or rather stay on 1.15

@ilhan007 ilhan007 added this to the 1.20.0 milestone Dec 4, 2023
@i832513
Copy link
Author

i832513 commented Dec 12, 2023

@ilhan007 ,
Can you please downport this to 1.17.X because we are using that version in production right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue is a bug in the code High Prio SAP SF TOPIC Core
Projects
Status: Completed
Development

Successfully merging a pull request may close this issue.

5 participants