Skip to content

Commit

Permalink
RN local-cli: format and @flow filterPlatformAssetScales
Browse files Browse the repository at this point in the history
Summary: Preparation before using that from elsewhere that's already typed.

Reviewed By: cpojer

Differential Revision: D5678616

fbshipit-source-id: 01ef6e688241e07ae9ce6aba27c6d4980f6a688b
  • Loading branch information
Jean Lauliac authored and facebook-github-bot committed Aug 22, 2017
1 parent 4aae843 commit 0a4085b
Showing 1 changed file with 11 additions and 4 deletions.
15 changes: 11 additions & 4 deletions local-cli/bundle/filterPlatformAssetScales.js
Original file line number Diff line number Diff line change
Expand Up @@ -5,23 +5,30 @@
* This source code is licensed under the BSD-style license found in the
* LICENSE file in the root directory of this source tree. An additional grant
* of patent rights can be found in the PATENTS file in the same directory.
*
* @flow
* @format
*/

'use strict';

const ALLOWED_SCALES = {
ios: [1, 2, 3],
};

function filterPlatformAssetScales(platform, scales) {
function filterPlatformAssetScales(
platform: string,
scales: $ReadOnlyArray<number>,
): $ReadOnlyArray<number> {
const whitelist = ALLOWED_SCALES[platform];
if (!whitelist) {
return scales;
}
const result = scales.filter(scale => whitelist.indexOf(scale) > -1);
if (result.length === 0 && scales.length > 0) {
// No matching scale found, but there are some available. Ideally we don't want to
// be in this situation and should throw, but for now as a fallback let's just use
// the closest larger image
// No matching scale found, but there are some available. Ideally we don't
// want to be in this situation and should throw, but for now as a fallback
// let's just use the closest larger image
const maxScale = whitelist[whitelist.length - 1];
for (const scale of scales) {
if (scale > maxScale) {
Expand Down

0 comments on commit 0a4085b

Please sign in to comment.