summaryrefslogtreecommitdiffstats
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
* Merge commit 'b26b9425d700359d6ee68d45ac334fd1f2ab82d4' into remoteLalit Kansara2016-12-0615-131/+786
|\ | | | | | | | | | | | | Conflicts: src/com/android/packageinstaller/permission/ui/handheld/GrantPermissionsViewHandlerImpl.java Change-Id: Ia57d17750406754f1e19266f51f0115e3cf774ba
| * Support install once feature.Philip P. Moltmann2016-10-131-8/+18
| | | | | | | | | | | | | | Test: Simulated OK result from settings and triggered run-once workflow Fixes: 32105068 Change-Id: I9f6c3b56b09d520d2bc9c6eeb6c75f151e3f1aff
| * If the permission dialog would stick out, move it.Philip P. Moltmann2016-09-301-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | Also scroll the whole dialog if the screen is too small to for the dialog. Test: Executed steps in the bug. Dialog looks fine. Also toyed around with display scaling and font scaling to make sure dialog always looks ok (but might scroll) on very small displays. Fixes: 31442798 Change-Id: I74328afde38ec2170a14aaa8e77675ad2164460e (cherry picked from commit 3a5c306e5b6bf2285980aa34cb8eb00fbcf5ea2c)
| * Parse package before enabling unknown sourcesPhilip P. Moltmann2016-09-271-12/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So that when we return from the settings app we do not have distinguish between file and content-uri based install anymore Test: Manually went through the following workflows: - Install from file - Unknown sources is already enabled - Unknown sources is not enabled yet - Return 0 from settings - unknown sources was enabled - unknown sources was not enabled - Return -1 from settings - unknown sources was enabled - unknown sources was not enabled - Install from content URI - Unknown sources is already enabled - Unknown sources is not enabled yet - Return 0 from settings - unknown sources was enabled - unknown sources was not enabled - Return -1 from settings - unknown sources was enabled - unknown sources was not enabled Fixes: 31328285 Change-Id: I1d23f2ba9a835f7bffc59108565b286aeb675411
| * Calculate and apply 'installed size'Ryuki Nakamura2016-09-121-0/+9
| | | | | | | | | | Bug: 30056625 Change-Id: I1ffc7ba88ea5f99591b66c08e72ea07d60ab82d5
| * Resolve merge conflict from Ia3d8f54624801696eaec4f9216d7bbefb0b105a4.Philip P. Moltmann2016-09-011-1/+1
| | | | | | | | | | Bug: 30056625 Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
| * Read and apply 'packageName'Ryuki Nakamura2016-08-311-1/+4
| | | | | | | | | | Bug: 30056625 Change-Id: Ie8e46fc7a2c340fb04b64feb331af0d882fe9672
| * Merge "Fix the visibility for disabled apps in settings" into nyc-mr1-devLei Yu2016-08-301-0/+4
| |\
| | * Fix the visibility for disabled apps in settingsjackqdyulei2016-08-261-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 30770664 Test: Manual - 1. Disable one app 2. Go to Settings -> Apps -> menu -> App permissions 3. to check the visibility of that app. Don't show disabled app in the PermissionAppsFragment. Change-Id: I0d97f7011c13c9a78e988901285f10ca9e5b2cd8 (cherry picked from commit 92c2b41b91bffcc5ef1dbbe37ef66720753180ad)
| * | Fix the sorting bugs for system appsjackqdyulei2016-08-261-0/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bug: 30770664 Test: Manual - 1. Go to settings -> Apps -> Menu -> App permissions 2. Click one permissions folder. 3. Observe the sequence of apps. 4. Click the "show system" menu. 5. Observe the sequence of apps. Now the system apps and user apps are merged and sorted by alphabetic order. Change-Id: I45c933f9be30f138085f46f9515c38bf998bfb9a (cherry picked from commit b7ea973e7ad8e8c0a5dad420e0fc705efc92ec0b)
| * Fix an exception in PackageInstallerSvetoslav Ganov2016-08-231-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | It is apparently possible for an activity to get a configuration change callback when its view is no longer attached to the window manager. Check if a view is attached before removing it from the window manager. bug:30389560 Change-Id: I384401742eca61d140c7590eb5c67150a15da83b
| * Requested unused permissions reported as granted for app using none.Svet Ganov2016-08-111-0/+2
| | | | | | | | | | | | | | | | | | | | | | If the app uses no permissions and requested a runtime permission, the latter is reported granted while it is correctly not granted. Now we initialize everything as denied in case we exit eraly as the app does not use any permissions. bug:30747277 Change-Id: I08dbf086e7c3d5f66b6831c54910fbb4f0943388
| * Prevent "dont't ask again" being inpropoerly propagatedSvet Ganov2016-08-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | If the first permission of a several permission request is denied with prejudice the rest of the denied permissions in the sequence are also incorrectly denied with prejudice even if the user didn't select "don't ask again". bug:30653785 Change-Id: Ib942dd3c692d1c3a09dc1a64f2947ded0f193923
| * Remove lefover secure buttonSvetoslav Ganov2016-07-181-56/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | An incorrect fix for a security bug where apps with overlay access can redress the permission message added a secure button implementation that refuses interaction if the window is covered. While this was fixed in M branches we did not merge the change in NYC because we now hide all overlays. However, the app op which controls overlays allows privileged apps to bypass the app op. It is fine to have priv apps overlay stuff on top of the permissions UI except that the allow button show be clickable. This change removes the offending code. bug:29879858 Change-Id: I9d0b5e62b5ef029069e37f34dbcd90d4909d966b
| * Merge "Trigger transparency dialog only if restriction is set by admin." ↵TreeHugger Robot2016-07-141-5/+11
| |\ | | | | | | | | | into nyc-mr1-dev
| | * Trigger transparency dialog only if restriction is set by admin.Sudheer Shanka2016-07-121-5/+11
| | | | | | | | | | | | | | | Bug: 30104199 Change-Id: I97a16938c36b4dadc9167652f6297acf829366e9
| * | Hide toast windows when permissions UI shownSvetoslav Ganov2016-07-131-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When the permissions UI is showing we are hiding the system alrert windows that can overlay it in an attempt to redress the UI. It is however, also possible to use a toast window to overlay the permissions UI. This change makes sure we hide the toasts in addition to system alert windows when the permissions UI is showing. bug:30087538 Change-Id: Ifb9ecf4593ca24b628212f0737d574a537f0cccd
| * | Read and apply 'installLocation'Philip P. Moltmann2016-07-111-0/+8
| |/ | | | | | | | | | | | | | | | | | | ... when installing app via packageInstaller. This allows the apk to have some limited control over where it will be installed. Change-Id: Id5140e7d2a2e3b8dd59815a0febf5c05cee08fbf Fixes: 30056625
| * Do not allow downgrades of embedded microappsVinod Krishnan2016-07-062-4/+20
| | | | | | | | | | | | | | | | - flag is passed in from https://critique.corp.google.com/#review/126666577 Bug: 29641183 Change-Id: I1becbb446d824331430eb37d99bdbc7a36dd13db
| * Don't offer menu items it they are no-opSvet Ganov2016-06-292-13/+37
| | | | | | | | | | | | | | | | | | | | | | In the permission apps screen we have a menu item to show/hide system apps. It is possible that no system app uses the permission making the menu item do nothing. We don't add the menu item if no system app uses the permission. bug:29395895 Change-Id: I1756981b2f711fc7a886bd4227196fef1a3cea03
| * Only log (not crash) when activity is not found.Philip P. Moltmann2016-06-171-1/+6
| |\ | | | | | | | | | | | | | | | am: 87ec609c14 Change-Id: I07c7e57630269d9d37d8e10dfa49e59e88cec022
| * \ Merge \"Hide Status and Navigation Bar when uninstalling\" into nyc-devPhilip P. Moltmann2016-06-101-0/+8
| |\ \ | | | | | | | | | | | | | | | | | | | | am: 9c177f8d28 Change-Id: Ieac52df39c5481ea8c964be61d3a5a5c9bc024e3
| * \ \ Fix a crash in package installerSvet Ganov2016-06-092-2/+1
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | am: 869786e3e4 Change-Id: Ic1d8d6221194731a6c967f736a1352a905bba8b0
| * \ \ \ Merge "Show installer UI for user to confirm side-loading" into nyc-devSvetoslav Ganov2016-06-011-7/+1
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 6622140841 * commit '662214084178a687a32823decd4f42ba82313def': Show installer UI for user to confirm side-loading Change-Id: I017268c1d12c5fdfaea099e9428c3def1b728bdd
| * \ \ \ \ Merge "Format the template using HTML tags." into nyc-devPhilip P. Moltmann2016-05-251-11/+4
| |\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: e071d11592 * commit 'e071d1159277a00e0484dbf43899bc90ea27176c': Format the template using HTML tags. Change-Id: I1858bc3c1e6d6fb5f4b37a2ef4a287de635045ae
| * \ \ \ \ \ Finish package installer activity on a cancel resultSvetoslav Ganov2016-05-181-0/+3
| |\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 6fdd93b339 * commit '6fdd93b33927675a6a062f0b8fd00ef3866ac2f7': Finish package installer activity on a cancel result Change-Id: If34ab52b7c25cdbf69a06db5b4cf7e87b62e12bd
| * \ \ \ \ \ \ Merge "Add missing else statement" into nyc-devSvet Ganov2016-05-181-2/+2
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 622c5b563e * commit '622c5b563e1f2e7a78e64129f350387613161153': Add missing else statement Change-Id: I0e6e13bf0bbd368375bd924c496d1b8e011cbf19
| * \ \ \ \ \ \ \ Merge "Take advantage of new MotionEvent flag to prevent tapjacking." into ↵Michael Wright2016-05-161-0/+56
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | nyc-dev am: 1be5a77aaa * commit '1be5a77aaa3bfba81063cbba38422dfe30c4ce40': Take advantage of new MotionEvent flag to prevent tapjacking. Change-Id: I416339e7bb12371480432699499871cf3c43bcab
| * \ \ \ \ \ \ \ \ Merge "Ensure app label properly bolded" into nyc-devSvetoslav Ganov2016-05-141-2/+2
| |\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 55dbb25ae3 * commit '55dbb25ae341353ef10312d89a248d4e658be9ee': Ensure app label properly bolded Change-Id: If437d0944f8dc45107d67252ccf1f17e8c748ca3
| * \ \ \ \ \ \ \ \ \ Merge "Use badged icons for permissions." into nyc-devPhilip P. Moltmann2016-05-131-10/+12
| |\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 988f3bcb21 * commit '988f3bcb21b7026cfab89d22ef912f98c9c9f8a2': Use badged icons for permissions. Change-Id: I922e3df467d31be00c80128c91330b381853ce70
| * \ \ \ \ \ \ \ \ \ \ Merge "Finish whole installation on cancel dialog." into nyc-devPhilip P. Moltmann2016-05-131-1/+1
| |\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: f682d8970c * commit 'f682d8970cc10e9943232cde1888d214637e610e': Finish whole installation on cancel dialog. Change-Id: I785f39748871178c428330c70428d2a8b4979f78
| * \ \ \ \ \ \ \ \ \ \ \ Merge "Fine tune UI of package installer." into nyc-devPhilip P. Moltmann2016-05-121-1/+3
| |\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: 51b7ac8dc1 * commit '51b7ac8dc1b2e6ccb2d3af2f1c1d84b529977e3a': Fine tune UI of package installer. Change-Id: I0b2407e42d74ac3c15b3926f202b327de5561c9a
| * \ \ \ \ \ \ \ \ \ \ \ \ Fix app install flowSvetoslav Ganov2016-05-122-9/+25
| |\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | am: f63739bb3f * commit 'f63739bb3f5de9927171b84630ec5872ba85fa6c': Fix app install flow Change-Id: I37bdfd9f7e005d57eb90e46f87f4cabcfaa6519c
| * | | | | | | | | | | | | | Update Wearable install logic to use stream APIAnthony Hugh2016-05-095-32/+624
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The default directory for PackageInstaller moved from /data/user/foo to /data/user_de/foo. Not sure on the exact reason why, but it has something to do with the file based encryption changes in N. This was causing the temp APK file that we were passing to PackageManager for installs to be in an invalid location. This resulted in failed installations. This is OK though because we never really wanted to use a temporary file. We were using it due to limitations in the PackageInstaller APIs. Since implementation, new APIs have been added to allow us to pass a stream through instead. This CL updates the install logic to use the stream API instead. Because this code path is being deprecated and replaced, I've opted to do the bare minimum needed to keep this code operational. I've also borrowed installation code from Wearsky/Finsky codebase to make it easier and more reliable. BUG: 28021618 Change-Id: Ia245abf56649e350e99cf48d2b503906d3cd855c
* | | | | | | | | | | | | | | Disable permission auto enable in case only disable it under group permissionArthur Shuai2016-10-202-6/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The permission of dial out will be auto enabled if only disable it under call group permission. This change disable auto enable permission in this case. Change-Id: Ibe8c9064fcd4fdfd33da69d68c145a5108a484d2 CRs-Fixed:1079155
* | | | | | | | | | | | | | | Merge "Make storage permission only show one configration."Linux Build Service Account2016-10-091-18/+21
|\ \ \ \ \ \ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | | | | | | Make storage permission only show one configration.Arthur Shuai2016-09-291-18/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update application permission setting,the storage permission only show one configration. Change-Id: Ic53e6a165e759a401bd8e1e3037ce795a478c3b3 CRs-Fixed: 1066060
* | | | | | | | | | | | | | | | Grant group permissions instead of single permission.Arthur Shuai2016-09-291-139/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In grant process, when require permission, will grant all group permissions instead of single permission. CRs-Fixed: 1066060 Change-Id: Iafe1d308a6ecb301da1c00450838a87344fc7f5f
* | | | | | | | | | | | | | | | Add dialog when revoke one permissionJiao Yuan2016-09-201-8/+24
|/ / / / / / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When revoke one permission in the settings, add a warning dialog to users. Change-Id: I157a4b0aab0e8d4f9719b4ff4d68f7eb11922ec3
* | | | | | | | | | | | | | | Show every single permission only for platform permission groupJiao Yuan2016-09-081-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Package installer stopped when tap email permissions, because email defines an additional permission, but package installer can not find its group,so it throws null point exception, so show every single permission only for platform permission group. Change-Id: Ib56b4b4b6fef2ec112321787d34c35c211652a60
* | | | | | | | | | | | | | | PackageInstaller: Add macro for CTA codec_jyuan2016-08-241-3/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | UpdateEveryPermissionPreference func is only for CTA test. Add persist.sys.strict_op_enable for CTA code, otherwise will cause nullPointerException. Change-Id: I3ad2019e245e8971e22657edaa76c7348d485e34
* | | | | | | | | | | | | | | PackageInstaller: Enable permission controlJiao Yuan2016-08-244-52/+235
| |_|_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Enable permission control for each permission. Change-Id: Id045e432536233f0924cc7497a5dd528066da701
* | | | | | | | | | | | | | Only log (not crash) when activity is not found.Philip P. Moltmann2016-06-161-1/+6
| |_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixes: 29260288 Change-Id: Iaa309f362ae6a296f7ba365e418d4a6f55b23f0c
* | | | | | | | | | | | | Merge "Hide Status and Navigation Bar when uninstalling" into nyc-devPhilip P. Moltmann2016-06-101-0/+8
|\ \ \ \ \ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | |
| * | | | | | | | | | | | Hide Status and Navigation Bar when uninstallingPhilip P. Moltmann2016-06-021-0/+8
| | |_|_|_|_|_|_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... at least unless we want to show a progress dialog. Fixes: 29094372 Change-Id: I45e0bbf32355a93635c773c526151b849bc1e8a3
* / | | | | | | | | | | Fix a crash in package installerSvet Ganov2016-06-082-2/+1
|/ / / / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The first bug was that we were starting the install before having the package info for the installed app. The second was that we were calling the permission review UI despite we were there before the user selected to install. i.e. the user already reviewed the permissions. bug:29197887 Change-Id: I2fac4d014dde7f7744a6b831c2b0ff510e29e8e8
* | | | | | | | | | / Show installer UI for user to confirm side-loadingSvetoslav Ganov2016-06-011-7/+1
| |_|_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change fixes a regression where we stopped showing a confirmation UI for an app side-loading. bug:28551937 Change-Id: Ia2114d273473d6e00696f9de8c523190a1b444ad
* | | | | | | | | | Format the template using HTML tags.Philip P. Moltmann2016-05-231-11/+4
| |_|_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Manual formatting is to complicated to do. Fixes: 28852429 Change-Id: Ibab723b82a2b14ad94a3038dffccc96f86ba8ee4
* | | | | | | | | Finish package installer activity on a cancel resultSvetoslav Ganov2016-05-181-0/+3
| |_|_|_|_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We made a change to the PackageInstaller to start settings for a result to allow partners to return OK to continue the install flow, otherwise requiring the user to start install again (the desired behavior on Nexus). This change finishes the package installer activity if settings does not return success to continue the install flow. bug:28838075 Change-Id: I4ff1b1a713d46a8584baf66f379ab070226c125f
* | | | | | | | Merge "Add missing else statement" into nyc-devSvetoslav Ganov2016-05-181-2/+2
|\ \ \ \ \ \ \ \ | |_|_|_|_|_|_|/ |/| | | | | | |