[Android] fix border style without borderRadius #6789

skv-headless · 2016-04-03T21:33:06Z

closes #6721 closes #3159
before(without borderRadius)

after

facebook-github-bot · 2016-04-03T21:33:18Z

By analyzing the blame information on this pull request, we identified @spicyj, @mkonicek and @vjeux to be potential reviewers.

chirag04 · 2016-04-03T22:42:56Z

will also fix #3159.

I haven't tested this change tho.

ide · 2016-04-04T01:45:46Z

ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewBackgroundDrawable.java

@@ -95,7 +95,7 @@

  @Override
  public void draw(Canvas canvas) {
-    if ((!CSSConstants.isUndefined(mBorderRadius) && mBorderRadius > 0) || mBorderCornerRadii != null) {
+    if ((!CSSConstants.isUndefined(mBorderRadius) && mBorderRadius > 0) || mBorderCornerRadii != null || mBorderStyle != null) {


Can you check for "solid" too?

skv-headless · 2016-04-04T15:04:30Z

@ide please don't merge. In that solution there is problem with changing border style. I need more time to explore code.

facebook-github-bot · 2016-04-04T21:04:42Z

@skv-headless updated the pull request.

skv-headless · 2016-04-04T21:22:50Z

@ide done. But variable names are not obvious. And I'm not sure about difference between drawRoundedBackgroundWithBorders and drawRectangularBackgroundWithBorders

skv-headless · 2016-04-06T13:25:58Z

@ide Do I need to explain something?

ide · 2016-04-06T21:08:35Z

ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewBackgroundDrawable.java

@@ -322,6 +327,7 @@ private int getFullBorderColor() {
  }

  private void drawRectangularBackgroundWithBorders(Canvas canvas) {
+    updatePathEffectForBorderStyle();


Can this call go above line 341?

mkonicek · 2016-04-07T14:08:11Z

@kmagiera What do you think?

facebook-github-bot · 2016-04-11T09:21:14Z

@skv-headless updated the pull request.

skv-headless · 2016-04-11T09:24:11Z

Do I need to do something else? My changes are pretty straightforward.

mkonicek · 2016-04-11T10:11:18Z

@skv-headless This is waiting for a review by @kmagiera who I believe has context on this part of the codebase. If he doesn't get back to you in a few days please ping me.

facebook-github-bot · 2016-04-16T18:04:15Z

@skv-headless updated the pull request.

satya164 · 2016-04-16T21:46:00Z

ping @mkonicek

mkonicek · 2016-04-21T14:40:57Z

Examples/UIExplorer/ViewExample.js

@@ -46,7 +46,6 @@ var ViewBorderStyleExample = React.createClass({
        <View>
          <View style={{
            borderWidth: 1,
-            borderRadius: 5,


Why remove the border radius from the example?

To show that this patch is working.

mkonicek · 2016-04-21T14:47:54Z

Thanks for the fix! Looks good, just some small nits.

facebook-github-bot · 2016-04-25T08:21:14Z

@skv-headless updated the pull request.

skv-headless · 2016-04-25T08:21:52Z

ReactAndroid/src/main/java/com/facebook/react/views/view/ReactViewBackgroundDrawable.java

+    boolean roundedBorders = mBorderCornerRadii != null ||
+        (!CSSConstants.isUndefined(mBorderRadius) && mBorderRadius > 0);
+
+    if ((mBorderStyle == null || mBorderStyle == BorderStyle.SOLID) && !roundedBorders) {


I did this condition because I'm not sure how to call (mBorderStyle == null || mBorderStyle == BorderStyle.SOLID) this condition

skv-headless · 2016-04-25T08:23:40Z

@mkonicek I've updated PR. Sorry for delay I was really busy 😫

facebook-github-bot · 2016-04-25T08:25:19Z

@skv-headless updated the pull request.

call drawRoundedBackgroundWithBorders if borderStyle set

ghost · 2016-04-26T17:00:31Z

@skv-headless do you have any updates for this pull request? It's been a while so we wanted to check in and see if you've looked at the requested changes.

skv-headless · 2016-04-27T11:56:15Z

@mkonicek ping

mkonicek · 2016-04-29T11:05:49Z

No problem @skv-headless, sorry on the delay on my side too :)

@facebook-github-bot shipit

ghost · 2016-04-29T11:07:48Z

Thanks for importing. If you are an FB employee go to Phabricator to review.

bestander · 2016-04-29T21:47:44Z

@mkonicek @skv-headless this change broke ObjC tests.
Please consider Travis build status https://travis-ci.org/facebook/react-native/builds/125507502

Considering that it is a long weekend ahead of us we don't want to have a red trunk I'll do a revert.
Please try one more time with Obj-C tests fixed :)

bestander · 2016-04-29T22:36:16Z

see #7303

janicduplessis · 2016-04-30T00:08:57Z

Is it documented anywhere how to update the snapshot tests? I know @grabbou had to do it a few days ago maybe he can help.

bestander · 2016-04-30T00:16:55Z

I think it is not documented but it is relatively straight forward.
You just need to run the failing test with XCode.
Every snapshot test has a setting "generate" or something.
Setting it to true would generate and replace the old snapshot with the new
one.
If you can't figure it out I'll help out next week :)

On Sat, Apr 30, 2016 at 1:10 AM, Janic Duplessis notifications@github.com
wrote:

Is it documented anywhere how to update the snapshot tests? I know
@grabbou https://github.com/grabbou had to do it a few days ago maybe
he can help.

—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
#6789 (comment)

grabbou · 2016-04-30T08:21:51Z

@janicduplessis thanks for the reminder, I am on that ;)

Summary: closes facebook#6721 closes facebook#3159 before(without borderRadius) ![bug](https://cloud.githubusercontent.com/assets/1488195/14235087/fcf79f8a-f9eb-11e5-9d44-7ab1d131be24.jpg) after ![fix](https://cloud.githubusercontent.com/assets/1488195/14235067/8cf128b4-f9eb-11e5-8170-ad3a146d6104.jpg) Closes facebook#6789 Differential Revision: D3240657 Pulled By: mkonicek fb-gh-sync-id: 4eb8f72d7278d880297fb73653ef530719af3d5b fbshipit-source-id: 4eb8f72d7278d880297fb73653ef530719af3d5b

closes facebook#6789

Bumps [axios](https://github.com/axios/axios) from 1.7.7 to 1.8.3. <details> <summary>Release notes</summary> <p><em>Sourced from <a href="https://github.com/axios/axios/releases">axios's releases</a>.</em></p> <blockquote> <h2>Release v1.8.3</h2> <h2>Release notes:</h2> <h3>Bug Fixes</h3> <ul> <li>add missing type for allowAbsoluteUrls (<a href="https://redirect.github.com/axios/axios/issues/6818">#6818</a>) (<a href="https://github.com/axios/axios/commit/10fa70ef14fe39558b15a179f0e82f5f5e5d11b2">10fa70e</a>)</li> <li><strong>xhr/fetch:</strong> pass <code>allowAbsoluteUrls</code> to <code>buildFullPath</code> in <code>xhr</code> and <code>fetch</code> adapters (<a href="https://redirect.github.com/axios/axios/issues/6814">#6814</a>) (<a href="https://github.com/axios/axios/commit/ec159e507bdf08c04ba1a10fe7710094e9e50ec9">ec159e5</a>)</li> </ul> <h3>Contributors to this release</h3> <ul> <li> <a href="https://github.com/Electroid" title="+6/-0 ([facebook#6811](axios/axios#6811) )">Ashcon Partovi</a></li> <li> <a href="https://github.com/StefanBRas" title="+4/-0 ([facebook#6818](axios/axios#6818) )">StefanBRas</a></li> <li> <a href="https://github.com/mhassan1" title="+2/-2 ([facebook#6814](axios/axios#6814) )">Marc Hassan</a></li> </ul> <h2>Release v1.8.2</h2> <h2>Release notes:</h2> <h3>Bug Fixes</h3> <ul> <li><strong>http-adapter:</strong> add allowAbsoluteUrls to path building (<a href="https://redirect.github.com/axios/axios/issues/6810">#6810</a>) (<a href="https://github.com/axios/axios/commit/fb8eec214ce7744b5ca787f2c3b8339b2f54b00f">fb8eec2</a>)</li> </ul> <h3>Contributors to this release</h3> <ul> <li> <a href="https://github.com/lexcorp16" title="+1/-1 ([facebook#6810](axios/axios#6810) )">Fasoro-Joseph Alexander</a></li> </ul> <h2>Release v1.8.1</h2> <h2>Release notes:</h2> <h3>Bug Fixes</h3> <ul> <li><strong>utils:</strong> move <code>generateString</code> to platform utils to avoid importing crypto module into client builds; (<a href="https://redirect.github.com/axios/axios/issues/6789">#6789</a>) (<a href="https://github.com/axios/axios/commit/36a5a620bec0b181451927f13ac85b9888b86cec">36a5a62</a>)</li> </ul> <h3>Contributors to this release</h3> <ul> <li> <a href="https://github.com/DigitalBrainJS" title="+51/-47 ([facebook#6789](axios/axios#6789) )">Dmitriy Mozgovoy</a></li> </ul> <h2>Release v1.8.0</h2> <h2>Release notes:</h2> <h3>Bug Fixes</h3> <ul> <li><strong>examples:</strong> application crashed when navigating examples in browser (<a href="https://redirect.github.com/axios/axios/issues/5938">#5938</a>) (<a href="https://github.com/axios/axios/commit/1260ded634ec101dd5ed05d3b70f8e8f899dba6c">1260ded</a>)</li> <li>missing word in SUPPORT_QUESTION.yml (<a href="https://redirect.github.com/axios/axios/issues/6757">#6757</a>) (<a href="https://github.com/axios/axios/commit/1f890b13f2c25a016f3c84ae78efb769f244133e">1f890b1</a>)</li> <li><strong>utils:</strong> replace getRandomValues with crypto module (<a href="https://redirect.github.com/axios/axios/issues/6788">#6788</a>) (<a href="https://github.com/axios/axios/commit/23a25af0688d1db2c396deb09229d2271cc24f6c">23a25af</a>)</li> </ul> <h3>Features</h3> <ul> <li>Add config for ignoring absolute URLs (<a href="https://redirect.github.com/axios/axios/issues/5902">#5902</a>) (<a href="https://redirect.github.com/axios/axios/issues/6192">#6192</a>) (<a href="https://github.com/axios/axios/commit/32c7bcc0f233285ba27dec73a4b1e81fb7a219b3">32c7bcc</a>)</li> </ul> <h3>Reverts</h3> <ul> <li>Revert "chore: expose fromDataToStream to be consumable (<a href="https://redirect.github.com/axios/axios/issues/6731">#6731</a>)" (<a href="https://redirect.github.com/axios/axios/issues/6732">#6732</a>) (<a href="https://github.com/axios/axios/commit/1317261125e9c419fe9f126867f64d28f9c1efda">1317261</a>), closes <a href="https://redirect.github.com/axios/axios/issues/6731">#6731</a> <a href="https://redirect.github.com/axios/axios/issues/6732">#6732</a></li> </ul>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Changelog</summary> <p><em>Sourced from <a href="https://github.com/axios/axios/blob/v1.x/CHANGELOG.md">axios's changelog</a>.</em></p> <blockquote> <h2><a href="https://github.com/axios/axios/compare/v1.8.2...v1.8.3">1.8.3</a> (2025-03-10)</h2> <h3>Bug Fixes</h3> <ul> <li>add missing type for allowAbsoluteUrls (<a href="https://redirect.github.com/axios/axios/issues/6818">#6818</a>) (<a href="https://github.com/axios/axios/commit/10fa70ef14fe39558b15a179f0e82f5f5e5d11b2">10fa70e</a>)</li> <li><strong>xhr/fetch:</strong> pass <code>allowAbsoluteUrls</code> to <code>buildFullPath</code> in <code>xhr</code> and <code>fetch</code> adapters (<a href="https://redirect.github.com/axios/axios/issues/6814">#6814</a>) (<a href="https://github.com/axios/axios/commit/ec159e507bdf08c04ba1a10fe7710094e9e50ec9">ec159e5</a>)</li> </ul> <h3>Contributors to this release</h3> <ul> <li> <a href="https://github.com/Electroid" title="+6/-0 ([facebook#6811](axios/axios#6811) )">Ashcon Partovi</a></li> <li> <a href="https://github.com/StefanBRas" title="+4/-0 ([facebook#6818](axios/axios#6818) )">StefanBRas</a></li> <li> <a href="https://github.com/mhassan1" title="+2/-2 ([facebook#6814](axios/axios#6814) )">Marc Hassan</a></li> </ul> <h2><a href="https://github.com/axios/axios/compare/v1.8.1...v1.8.2">1.8.2</a> (2025-03-07)</h2> <h3>Bug Fixes</h3> <ul> <li><strong>http-adapter:</strong> add allowAbsoluteUrls to path building (<a href="https://redirect.github.com/axios/axios/issues/6810">#6810</a>) (<a href="https://github.com/axios/axios/commit/fb8eec214ce7744b5ca787f2c3b8339b2f54b00f">fb8eec2</a>)</li> </ul> <h3>Contributors to this release</h3> <ul> <li> <a href="https://github.com/lexcorp16" title="+1/-1 ([facebook#6810](axios/axios#6810) )">Fasoro-Joseph Alexander</a></li> </ul> <h2><a href="https://github.com/axios/axios/compare/v1.8.0...v1.8.1">1.8.1</a> (2025-02-26)</h2> <h3>Bug Fixes</h3> <ul> <li><strong>utils:</strong> move <code>generateString</code> to platform utils to avoid importing crypto module into client builds; (<a href="https://redirect.github.com/axios/axios/issues/6789">#6789</a>) (<a href="https://github.com/axios/axios/commit/36a5a620bec0b181451927f13ac85b9888b86cec">36a5a62</a>)</li> </ul> <h3>Contributors to this release</h3> <ul> <li> <a href="https://github.com/DigitalBrainJS" title="+51/-47 ([facebook#6789](axios/axios#6789) )">Dmitriy Mozgovoy</a></li> </ul> <h1><a href="https://github.com/axios/axios/compare/v1.7.9...v1.8.0">1.8.0</a> (2025-02-25)</h1> <h3>Bug Fixes</h3> <ul> <li><strong>examples:</strong> application crashed when navigating examples in browser (<a href="https://redirect.github.com/axios/axios/issues/5938">#5938</a>) (<a href="https://github.com/axios/axios/commit/1260ded634ec101dd5ed05d3b70f8e8f899dba6c">1260ded</a>)</li> <li>missing word in SUPPORT_QUESTION.yml (<a href="https://redirect.github.com/axios/axios/issues/6757">#6757</a>) (<a href="https://github.com/axios/axios/commit/1f890b13f2c25a016f3c84ae78efb769f244133e">1f890b1</a>)</li> <li><strong>utils:</strong> replace getRandomValues with crypto module (<a href="https://redirect.github.com/axios/axios/issues/6788">#6788</a>) (<a href="https://github.com/axios/axios/commit/23a25af0688d1db2c396deb09229d2271cc24f6c">23a25af</a>)</li> </ul> <h3>Features</h3> <ul> <li>Add config for ignoring absolute URLs (<a href="https://redirect.github.com/axios/axios/issues/5902">#5902</a>) (<a href="https://redirect.github.com/axios/axios/issues/6192">#6192</a>) (<a href="https://github.com/axios/axios/commit/32c7bcc0f233285ba27dec73a4b1e81fb7a219b3">32c7bcc</a>)</li> </ul>  </blockquote> <p>... (truncated)</p> </details> <details> <summary>Commits</summary> <ul> <li><a href="https://github.com/axios/axios/commit/39ec206483a89921732bdc8a5be67e350bfc23f0"><code>39ec206</code></a> chore(release): v1.8.3 (<a href="https://redirect.github.com/axios/axios/issues/6819">#6819</a>)</li> <li><a href="https://github.com/axios/axios/commit/10fa70ef14fe39558b15a179f0e82f5f5e5d11b2"><code>10fa70e</code></a> fix: add missing type for allowAbsoluteUrls (<a href="https://redirect.github.com/axios/axios/issues/6818">#6818</a>)</li> <li><a href="https://github.com/axios/axios/commit/7821ef9f5be2d62fbc3f01040d9df6f2225eb9d8"><code>7821ef9</code></a> docs: update readme to include bun install (<a href="https://redirect.github.com/axios/axios/issues/6811">#6811</a>)</li> <li><a href="https://github.com/axios/axios/commit/ec159e507bdf08c04ba1a10fe7710094e9e50ec9"><code>ec159e5</code></a> fix(xhr/fetch): pass <code>allowAbsoluteUrls</code> to <code>buildFullPath</code> in <code>xhr</code> and `fet...</li> <li><a href="https://github.com/axios/axios/commit/a9f7689b0c4b6d68c7f587c3aa376860da509d94"><code>a9f7689</code></a> chore(release): v1.8.2 (<a href="https://redirect.github.com/axios/axios/issues/6812">#6812</a>)</li> <li><a href="https://github.com/axios/axios/commit/fb8eec214ce7744b5ca787f2c3b8339b2f54b00f"><code>fb8eec2</code></a> fix(http-adapter): add allowAbsoluteUrls to path building (<a href="https://redirect.github.com/axios/axios/issues/6810">#6810</a>)</li> <li><a href="https://github.com/axios/axios/commit/98120457559e573024862e2925d56295a965ad7e"><code>9812045</code></a> chore(sponsor): update sponsor block (<a href="https://redirect.github.com/axios/axios/issues/6804">#6804</a>)</li> <li><a href="https://github.com/axios/axios/commit/72acf759373ef4e211d5299818d19e50e08c02f8"><code>72acf75</code></a> chore(sponsor): update sponsor block (<a href="https://redirect.github.com/axios/axios/issues/6794">#6794</a>)</li> <li><a href="https://github.com/axios/axios/commit/2e64afdff5c41e38284a6fb8312f2745072513a1"><code>2e64afd</code></a> chore(release): v1.8.1 (<a href="https://redirect.github.com/axios/axios/issues/6800">#6800</a>)</li> <li><a href="https://github.com/axios/axios/commit/36a5a620bec0b181451927f13ac85b9888b86cec"><code>36a5a62</code></a> fix(utils): move <code>generateString</code> to platform utils to avoid importing crypto...</li> <li>Additional commits viewable in <a href="https://github.com/axios/axios/compare/v1.7.7...v1.8.3">compare view</a></li> </ul> </details> <br /> [![Dependabot compatibility score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=axios&package-manager=npm_and_yarn&previous-version=1.7.7&new-version=1.8.3)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores) Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting `@dependabot rebase`. [//]: # (dependabot-automerge-start) [//]: # (dependabot-automerge-end) --- <details> <summary>Dependabot commands and options</summary> <br /> You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot show <dependency name> ignore conditions` will show all of the ignore conditions of the specified dependency - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself) You can disable automated security fix PRs for this repo from the [Security Alerts page](https://github.com/microsoft/react-native-macos/network/alerts). </details> Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Apr 3, 2016

skv-headless force-pushed the fix-border-style branch from 8106095 to 06ca452 Compare April 3, 2016 22:07

skv-headless changed the title ~~fix border style without borderRadius~~ [Android] fix border style without borderRadius Apr 3, 2016

ide reviewed Apr 4, 2016
View reviewed changes

skv-headless force-pushed the fix-border-style branch from 06ca452 to b0521f8 Compare April 4, 2016 21:04

skv-headless force-pushed the fix-border-style branch from b0521f8 to be11548 Compare April 6, 2016 13:25

ide reviewed Apr 6, 2016
View reviewed changes

skv-headless force-pushed the fix-border-style branch 2 times, most recently from 8f3d0fd to 2d9bab6 Compare April 11, 2016 09:20

skv-headless force-pushed the fix-border-style branch from 2d9bab6 to 76dcc31 Compare April 16, 2016 18:03

grabbou assigned mkonicek Apr 20, 2016

mkonicek reviewed Apr 21, 2016
View reviewed changes

mkonicek added the GH Review: needs-revision label Apr 21, 2016

skv-headless force-pushed the fix-border-style branch from 76dcc31 to b04b10f Compare April 25, 2016 08:20

skv-headless reviewed Apr 25, 2016
View reviewed changes

skv-headless force-pushed the fix-border-style branch from b04b10f to cc7a78f Compare April 25, 2016 08:24

fix border style without borderRadius

cc7a78f

call drawRoundedBackgroundWithBorders if borderStyle set

ghost added GH Review: accepted Import Started This pull request has been imported. This does not imply the PR has been approved. and removed GH Review: needs-revision labels Apr 29, 2016

ghost closed this in 58876d5 Apr 29, 2016

skv-headless mentioned this pull request May 4, 2016

Revert "Revert "fix border style without borderRadius"" #7389

Closed

waquidvp mentioned this pull request Dec 27, 2017

borderStyle cannot working (v0.47.2 on Android) #17344

Closed

react-one pushed a commit to react-one/react-native that referenced this pull request Sep 24, 2021

fix: initialize height and width to zero if undefined

3df65e2

closes facebook#6789

This pull request was closed.

[Android] fix border style without borderRadius #6789

[Android] fix border style without borderRadius #6789

Uh oh!

Conversation

skv-headless commented Apr 3, 2016

Uh oh!

facebook-github-bot commented Apr 3, 2016

Uh oh!

chirag04 commented Apr 3, 2016

Uh oh!

ide Apr 4, 2016

Choose a reason for hiding this comment

Uh oh!

skv-headless commented Apr 4, 2016

Uh oh!

facebook-github-bot commented Apr 4, 2016

Uh oh!

skv-headless commented Apr 4, 2016

Uh oh!

skv-headless commented Apr 6, 2016

Uh oh!

ide Apr 6, 2016

Choose a reason for hiding this comment

Uh oh!

mkonicek commented Apr 7, 2016

Uh oh!

facebook-github-bot commented Apr 11, 2016

Uh oh!

skv-headless commented Apr 11, 2016

Uh oh!

mkonicek commented Apr 11, 2016

Uh oh!

facebook-github-bot commented Apr 16, 2016

Uh oh!

satya164 commented Apr 16, 2016

Uh oh!

mkonicek Apr 21, 2016

Choose a reason for hiding this comment

Uh oh!

skv-headless Apr 23, 2016

Choose a reason for hiding this comment

Uh oh!

mkonicek commented Apr 21, 2016

Uh oh!

facebook-github-bot commented Apr 25, 2016

Uh oh!

skv-headless Apr 25, 2016

Choose a reason for hiding this comment

Uh oh!

skv-headless commented Apr 25, 2016

Uh oh!

facebook-github-bot commented Apr 25, 2016

Uh oh!

ghost commented Apr 26, 2016

Uh oh!

skv-headless commented Apr 27, 2016

Uh oh!

mkonicek commented Apr 29, 2016

Uh oh!

ghost commented Apr 29, 2016

Uh oh!

bestander commented Apr 29, 2016

Uh oh!

bestander commented Apr 29, 2016

Uh oh!

janicduplessis commented Apr 30, 2016

Uh oh!

bestander commented Apr 30, 2016

Uh oh!

grabbou commented Apr 30, 2016

Uh oh!

Uh oh!