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

[CP] Null exception fix #113550

Closed
justinmc opened this issue Oct 17, 2022 · 9 comments
Closed

[CP] Null exception fix #113550

justinmc opened this issue Oct 17, 2022 · 9 comments
Assignees
Labels
cp: approved Approved cherry-pick request cp: review Cherry-picks in the review queue

Comments

@justinmc
Copy link
Contributor

justinmc commented Oct 17, 2022

issue_link

#111095

Commit Hash

d728539

Target

stable

pr_link

#113553

Impacted Users

Android apps with text editing

Impact Description

A null exception is thrown for users of text editing on Android.

Workaround

  1. Patch the Flutter framework directly with this one line fix: https://github.com/flutter/flutter/pull/107683/files#diff-f5f93c879cef9a102adbf148583de9f3d9a05f9678fdcad7bde106f62200bb34

or

  1. Upgrade to master or any release that includes the fix PR (Remove exclamation marks #107683).

Risk

medium

Test Coverage

yes

Validation Steps

If this test passes, then the fix works:

testWidgets('onAppPrivateCommand does not throw', (WidgetTester tester) async {

@justinmc justinmc added the cp: review Cherry-picks in the review queue label Oct 17, 2022
@itsjustkevin
Copy link
Contributor

@HansMuller could you review this cherry pick please?

@aprzedecki
Copy link

Would be great to include it in 3.4.0 release.

@itsjustkevin
Copy link
Contributor

@LongCatIsLooong could you take a look at this cherry-pick request?

@absar
Copy link

absar commented Oct 24, 2022

Can it be prioritized, it's regressing allover

@HansMuller
Copy link
Contributor

LGTM

@CaseyHillers CaseyHillers added the cp: approved Approved cherry-pick request label Oct 28, 2022
@CaseyHillers CaseyHillers self-assigned this Oct 31, 2022
@justinmc
Copy link
Contributor Author

justinmc commented Nov 1, 2022

Can this be closed now that the PR is merged or is the process to wait? #113553

@CaseyHillers
Copy link
Contributor

Can this be closed now that the PR is merged or is the process to wait? #113553

We wait until it has officially shipped to users. I expect us to remain on schedule, and ship this tomorrow (11/02) at 10 AM PST. Afterwards, I'll close the CP requests that were merged.

The reason we do this is occasionally a breakage is revealed in CI, and we have to revert a CP.

@CaseyHillers
Copy link
Contributor

This was released in 3.3.7

@github-actions
Copy link

This thread has been automatically locked since there has not been any recent activity after it was closed. If you are still experiencing a similar issue, please open a new bug, including the output of flutter doctor -v and a minimal reproduction of the issue.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 21, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cp: approved Approved cherry-pick request cp: review Cherry-picks in the review queue
Projects
None yet
Development

No branches or pull requests

6 participants