Update linter

2020-04-16 10:56:46 +02:00
parent 19e8bfdb37
commit 30af13f793
53 changed files with 92 additions and 97 deletions
--- a/app/src/actions/Charts.ts
+++ b/app/src/actions/Charts.ts
@@ -67,7 +67,7 @@ export const addChart = (chartParameters: ChartParameters) => async (
  const chartExists = Boolean(
    getState()
      .charts.get('charts')
-      .find((chart) => chart.topic === chartParameters.topic && chart.dotPath === chartParameters.dotPath)
+      .find(chart => chart.topic === chartParameters.topic && chart.dotPath === chartParameters.dotPath)
  )
  if (chartExists) {
    dispatch(showNotification('Already added'))
--- a/app/src/actions/Connection.ts
+++ b/app/src/actions/Connection.ts
@@ -20,7 +20,7 @@ export const connect = (options: MqttOptions, connectionId: string) => (
  const event = makeConnectionStateEvent(connectionId)
  const host = url.parse(options.url).hostname

-  rendererEvents.subscribe(event, (dataSourceState) => {
+  rendererEvents.subscribe(event, dataSourceState => {
    if (dataSourceState.connected) {
      const didReconnect = Boolean(getState().connection.tree)
      if (!didReconnect) {
--- a/app/src/actions/ConnectionManager.ts
+++ b/app/src/actions/ConnectionManager.ts
@@ -146,7 +146,7 @@ export const deleteConnection = (connectionId: string) => (dispatch: Dispatch<an
  const connectionIds = Object.keys(getState().connectionManager.connections)
  const connectionIdLocation = connectionIds.indexOf(connectionId)

-  const remainingIds = connectionIds.filter((id) => id !== connectionId)
+  const remainingIds = connectionIds.filter(id => id !== connectionId)
  const nextSelectedConnectionIndex = Math.min(remainingIds.length - 1, connectionIdLocation)
  const nextSelectedConnection = remainingIds[nextSelectedConnectionIndex]

--- a/app/src/actions/Global.ts
+++ b/app/src/actions/Global.ts
@@ -22,7 +22,7 @@ export const toggleSettingsVisibility = () => (dispatch: Dispatch<any>) => {
 }

 export const requestConfirmation = (title: string, inquiry: string) => (dispatch: Dispatch<any>) => {
-  return new Promise((resolve) => {
+  return new Promise(resolve => {
    const confirmationRequest = {
      title,
      inquiry,
--- a/app/src/actions/clearTopic.ts
+++ b/app/src/actions/clearTopic.ts
@@ -40,8 +40,8 @@ export const clearTopic = (topic: q.TreeNode<any>, recursive: boolean) => async
  const publishEvent = makePublishEvent(connectionId)

  topicsForPurging
-    .filter((t) => t.path() !== '' && t.hasMessage())
-    .map((t) => t.path())
+    .filter(t => t.path() !== '' && t.hasMessage())
+    .map(t => t.path())
    .forEach((path, idx) => {
      const mqttMessage = {
        topic: path,
--- a/app/src/actions/visibleTreeTraversal.ts
+++ b/app/src/actions/visibleTreeTraversal.ts
@@ -82,7 +82,7 @@ function findNextNodeUpward(
  }

  const neighborNodes = sortedNodes(settings, parent)
-  const nodeIdx = neighborNodes.findIndex((n) => n.path() === treeNode.path())
+  const nodeIdx = neighborNodes.findIndex(n => n.path() === treeNode.path())
  if (nodeIdx === 0) {
    return parent
  }
@@ -125,7 +125,7 @@ function findNextNodeDownwardNeighbor(
  }

  const neighborNodes = sortedNodes(settings, parent).filter(isTreeNodeVisible)
-  const nodeIdx = neighborNodes.findIndex((n) => n.path() === treeNode.path())
+  const nodeIdx = neighborNodes.findIndex(n => n.path() === treeNode.path())
  const downwardNeighbor = neighborNodes[nodeIdx + 1]
  if (downwardNeighbor) {
    return downwardNeighbor
--- a/app/src/components/Chart/Chart.tsx
+++ b/app/src/components/Chart/Chart.tsx
@@ -28,7 +28,7 @@ export default withTheme(
  memo((props: Props) => {
    const [width, setWidth] = React.useState(300)
    const [tooltip, setTooltip] = React.useState<Tooltip | undefined>()
-    const detectResize = React.useCallback((newWidth) => setWidth(newWidth), [])
+    const detectResize = React.useCallback(newWidth => setWidth(newWidth), [])

    const hintFormatter = React.useCallback(
      (point: any) => [
--- a/app/src/components/Chart/effects/useCustomYDomain.tsx
+++ b/app/src/components/Chart/effects/useCustomYDomain.tsx
@@ -27,7 +27,7 @@ function domainForData(data: Array<Point>): [number, number] {
  let max = data[0].y
  let min = data[0].y

-  data.forEach((d) => {
+  data.forEach(d => {
    if (max < d.y) {
      max = d.y
    }
--- a/app/src/components/ChartPanel/ChartSettings/ColorSettings.tsx
+++ b/app/src/components/ChartPanel/ChartSettings/ColorSettings.tsx
@@ -31,7 +31,7 @@ function ColorSettings(props: {
  )

  const menuItems = React.useMemo(() => {
-    return colors.map((color) => (
+    return colors.map(color => (
      <MenuItem
        style={{ minWidth: '8em', minHeight: '36px', backgroundColor: color, textAlign: 'center' }}
        key={color}
--- a/app/src/components/ChartPanel/ChartSettings/InterpolationSettings.tsx
+++ b/app/src/components/ChartPanel/ChartSettings/InterpolationSettings.tsx
@@ -38,7 +38,7 @@ function InterpolationSettings(props: {
  }, [curves])

  const menuItems = React.useMemo(() => {
-    return curves.map((curve) => (
+    return curves.map(curve => (
      <MenuItem key={curve} onClick={callbacks[curve]} selected={props.chart.interpolation === curve}>
        <Typography variant="inherit">{curve.replace(/_/g, ' ')}</Typography>
      </MenuItem>
--- a/app/src/components/ChartPanel/ChartSettings/TimeRangeSettings.tsx
+++ b/app/src/components/ChartPanel/ChartSettings/TimeRangeSettings.tsx
@@ -58,7 +58,7 @@ function TimeRangeSettings(props: Props) {
      >
        <Typography>Chart data within a time interval</Typography>
        <div style={{ padding: '0 16px', width: '275px', textAlign: 'center' }}>
-          {ranges.map((r) => {
+          {ranges.map(r => {
            return (
              <Button
                style={{ margin: '4px', textTransform: 'none' }}
--- a/app/src/components/ChartPanel/ChartSettings/colors.ts
+++ b/app/src/components/ChartPanel/ChartSettings/colors.ts
@@ -39,7 +39,7 @@ export function colors() {
    indigo,
    yellow,
  ]
-    .map((color) => [color[200], color[500], color[700]])
+    .map(color => [color[200], color[500], color[700]])
    .reduce((a, b) => a.concat(b), [])
    .sort((a, b) => colorCompare(a, b))
  return colors
--- a/app/src/components/ChartPanel/index.tsx
+++ b/app/src/components/ChartPanel/index.tsx
@@ -63,7 +63,7 @@ function ChartPanel(props: Props) {
    }
  }, [chartsInView])

-  const charts = props.charts.map((chartParameters) => (
+  const charts = props.charts.map(chartParameters => (
    <CSSTransition
      key={`${chartParameters.topic}-${chartParameters.dotPath || ''}`}
      timeout={{ enter: 500, exit: 500 }}
--- a/app/src/components/ConnectionSetup/AdvancedConnectionSettings.tsx
+++ b/app/src/components/ConnectionSetup/AdvancedConnectionSettings.tsx
@@ -36,7 +36,7 @@ class ConnectionSettings extends React.Component<Props, State> {

  private renderSubscriptions() {
    const connection = this.props.connection
-    return connection.subscriptions.map((subscription) => (
+    return connection.subscriptions.map(subscription => (
      <Subscription
        deleteAction={() => this.props.managerActions.deleteSubscription(subscription, connection.id)}
        subscription={subscription}
--- a/app/src/components/ConnectionSetup/ProfileList/index.tsx
+++ b/app/src/components/ConnectionSetup/ProfileList/index.tsx
@@ -27,7 +27,7 @@ function ProfileList(props: Props) {
    }
    const indexDirection = dir === 'next' ? 1 : -1
    const connectionArray = Object.values(connections)
-    const selectedIndex = connectionArray.map((connection) => connection.id).indexOf(selected)
+    const selectedIndex = connectionArray.map(connection => connection.id).indexOf(selected)
    const nextConnection = connectionArray[selectedIndex + indexDirection]
    if (nextConnection) {
      actions.selectConnection(nextConnection.id)
@@ -48,7 +48,7 @@ function ProfileList(props: Props) {
  return (
    <List style={{ height: '100%' }} component="nav" subheader={createConnectionButton}>
      <div className={classes.list}>
-        {Object.values(connections).map((connection) => (
+        {Object.values(connections).map(connection => (
          <ConnectionItem connection={connection} key={connection.id} selected={selected === connection.id} />
        ))}
      </div>
--- a/app/src/components/Demo/ShowText.tsx
+++ b/app/src/components/Demo/ShowText.tsx
@@ -65,7 +65,7 @@ class Demo extends React.Component<{ classes: any }, State> {
    let keys: Array<any> = []
    if (this.state.keys.length > 0) {
      keys = this.state.keys
-        .map((key) => [<Key key={key} keyboardKey={key} />])
+        .map(key => [<Key key={key} keyboardKey={key} />])
        .reduce((prev, current) => {
          return [prev, '+' as any, current]
        })
--- a/app/src/components/Layout/ContentView.tsx
+++ b/app/src/components/Layout/ContentView.tsx
@@ -25,7 +25,7 @@ function ContentView(props: Props) {
    setDetectedHeight(newHeight)
  }, [])

-  const detectSidebarSize = React.useCallback((width) => {
+  const detectSidebarSize = React.useCallback(width => {
    setDetectedSidebarWidth(width)
  }, [])

--- a/app/src/components/Layout/SearchBar.tsx
+++ b/app/src/components/Layout/SearchBar.tsx
@@ -33,7 +33,7 @@ function SearchBar(props: {
    actions.settings.filterTopics(event.target.value)
  }, [])

-  useGlobalKeyEventHandler(undefined, (event) => {
+  useGlobalKeyEventHandler(undefined, event => {
    const isCharacter = event.key.length === 1
    const isModifierKey = event.metaKey || event.ctrlKey
    const isAllowedControlCharacter = event.keyCode === KeyCodes.backspace || event.keyCode === KeyCodes.delete
--- a/app/src/components/SettingsDrawer/Settings.tsx
+++ b/app/src/components/SettingsDrawer/Settings.tsx
@@ -143,7 +143,7 @@ class Settings extends React.PureComponent<Props, {}> {
  private renderAutoExpand() {
    const { classes, autoExpandLimit } = this.props

-    const limits = autoExpandLimitSet.map((limit) => (
+    const limits = autoExpandLimitSet.map(limit => (
      <MenuItem key={limit.limit} value={limit.limit}>
        {limit.limit < 10000 && limit.limit > 0 ? `≤ ${limit.limit} topics` : limit.name}
      </MenuItem>
--- a/app/src/components/SettingsDrawer/TimeLocale.tsx
+++ b/app/src/components/SettingsDrawer/TimeLocale.tsx
@@ -47,7 +47,7 @@ function TimeLocaleSettings(props: Props) {
        input={<Input name="time-locale" id="time-locale-label-placeholder" />}
        name="time-locale"
        className={classes.input}
-        renderValue={(v) => <span>{String(v)}</span>}
+        renderValue={v => <span>{String(v)}</span>}
        style={{ flex: '1' }}
      >
        {localeMenuItems}
--- a/app/src/components/Sidebar/CodeDiff/DiffCount.tsx
+++ b/app/src/components/Sidebar/CodeDiff/DiffCount.tsx
@@ -9,11 +9,9 @@ interface Props {
 }

 function changeAmount(props: Props) {
-  const additions = props.changes
-    .map((change) => (change.added === true ? change.count || 0 : 0))
-    .reduce((a, b) => a + b)
+  const additions = props.changes.map(change => (change.added === true ? change.count || 0 : 0)).reduce((a, b) => a + b)
  const deletions = props.changes
-    .map((change) => (change.removed === true ? change.count || 0 : 0))
+    .map(change => (change.removed === true ? change.count || 0 : 0))
    .reduce((a, b) => a + b)
  if (additions === 0 && deletions === 0) {
    return null
--- a/app/src/components/Sidebar/Publish/Publish.tsx
+++ b/app/src/components/Sidebar/Publish/Publish.tsx
@@ -30,7 +30,7 @@ function useHistory(): [Array<Message>, (topic: string, payload?: string) => voi
  const amendToHistory = useCallback(
    (topic: string, payload?: string) => {
      // Remove duplicates
-      let filteredHistory = history.filter((e) => e.payload !== payload || e.topic !== topic)
+      let filteredHistory = history.filter(e => e.payload !== payload || e.topic !== topic)
      filteredHistory = filteredHistory.slice(-7)
      setHistory([...filteredHistory, { topic, payload, sent: new Date() }])
    },
@@ -157,12 +157,9 @@ const PublishButton = (props: { publish: () => void }) => {
    [props.publish]
  )

-  const onFocus = useCallback(
-    (e: React.FocusEvent<HTMLElement>) => {
-      (e.relatedTarget as HTMLElement|null)?.focus()
-    },
-    []
-  )
+  const onFocus = useCallback((e: React.FocusEvent<HTMLElement>) => {
+    ;(e.relatedTarget as HTMLElement | null)?.focus()
+  }, [])

  return useMemo(
    () => (
--- a/app/src/components/Sidebar/Publish/PublishHistory.tsx
+++ b/app/src/components/Sidebar/Publish/PublishHistory.tsx
@@ -18,7 +18,7 @@ function PublishHistory(props: { history: Array<Message>; actions: typeof publis
  )

  return useMemo(() => {
-    const items = [...props.history].reverse().map((message) => ({
+    const items = [...props.history].reverse().map(message => ({
      key: sha1(message.topic + message.payload),
      title: message.topic,
      value: message.payload || '',
--- a/app/src/components/Sidebar/TopicPanel/Topic.tsx
+++ b/app/src/components/Sidebar/TopicPanel/Topic.tsx
@@ -33,9 +33,9 @@ class Topic extends React.PureComponent<Props, {}> {
    let key = 0
    const breadCrumps = node
      .branch()
-      .map((node) => node.sourceEdge)
-      .filter((edge) => Boolean(edge))
-      .map((edge) => [
+      .map(node => node.sourceEdge)
+      .filter(edge => Boolean(edge))
+      .map(edge => [
        <Button
          onClick={() => this.props.actions.selectTopic(edge!.target)}
          size="small"
--- a/app/src/components/TopicPlot.tsx
+++ b/app/src/components/TopicPlot.tsx
@@ -19,7 +19,7 @@ interface Props {
 function filterUsingTimeRange(startTime: number | undefined, data: Array<q.Message>) {
  if (startTime) {
    const threshold = new Date(Date.now() - startTime)
-    return data.filter((d) => d.received >= threshold)
+    return data.filter(d => d.received >= threshold)
  }

  return data
@@ -31,7 +31,7 @@ function nodeToHistory(startTime: number | undefined, history: q.MessageHistory)
      const value = message.value ? toPlottableValue(Base64Message.toUnicodeString(message.value)) : NaN
      return { x: message.received.getTime(), y: toPlottableValue(value) }
    })
-    .filter((data) => !isNaN(data.y as any)) as any
+    .filter(data => !isNaN(data.y as any)) as any
 }

 function nodeDotPathToHistory(startTime: number | undefined, history: q.MessageHistory, dotPath: string) {
@@ -46,7 +46,7 @@ function nodeDotPathToHistory(startTime: number | undefined, history: q.MessageH

      return { x: message.received.getTime(), y: toPlottableValue(value) }
    })
-    .filter((data) => !isNaN(data.y as any)) as any
+    .filter(data => !isNaN(data.y as any)) as any
 }

 function TopicPlot(props: Props) {
--- a/app/src/components/Tree/TreeNode/TreeNodeSubnodes.tsx
+++ b/app/src/components/Tree/TreeNode/TreeNodeSubnodes.tsx
@@ -48,7 +48,7 @@ function TreeNodeSubnodes(props: Props) {

  return useMemo(() => {
    const nodes = sortedNodes(props.settings, props.treeNode).slice(0, alreadyAdded)
-    const listItems = nodes.map((node) => {
+    const listItems = nodes.map(node => {
      return (
        <TreeNode
          key={`${node.hash()}-${props.filter}`}
--- a/app/src/components/UpdateNotifier.tsx
+++ b/app/src/components/UpdateNotifier.tsx
@@ -51,10 +51,10 @@ class UpdateNotifier extends React.PureComponent<Props, State> {
    this.state = { newerVersions: [] }

    const ownVersion = electron.remote.app.getVersion()
-    this.fetchReleases().then((releases) => {
+    this.fetchReleases().then(releases => {
      const newerVersions = releases
-        .filter((release) => this.allowPrereleaseIfOwnVersionIsBeta(release, ownVersion))
-        .filter((release) => compareVersions(release.tag_name, ownVersion) > 0)
+        .filter(release => this.allowPrereleaseIfOwnVersionIsBeta(release, ownVersion))
+        .filter(release => compareVersions(release.tag_name, ownVersion) > 0)
        .sort((a, b) => compareVersions(b.tag_name, a.tag_name))

      if (newerVersions.length > 0) {
@@ -145,7 +145,7 @@ class UpdateNotifier extends React.PureComponent<Props, State> {
      return null
    }
    const releaseNotes = this.state.newerVersions
-      .map((release) => `<p><h3>${release.tag_name}</h3><p/><p>${release.body_html}</p>`)
+      .map(release => `<p><h3>${release.tag_name}</h3><p/><p>${release.body_html}</p>`)
      .join('<hr />')

    return (
@@ -195,7 +195,7 @@ class UpdateNotifier extends React.PureComponent<Props, State> {
      return null
    }

-    return latestUpdate.assets.filter(this.assetForCurrentPlatform).map((asset) => (
+    return latestUpdate.assets.filter(this.assetForCurrentPlatform).map(asset => (
      <div>
        <Button className={this.props.classes.download} onClick={() => this.openUrl(asset.browser_download_url)}>
          <CloudDownload />
--- a/app/src/components/helper/ConnectionHealthIndicator.tsx
+++ b/app/src/components/helper/ConnectionHealthIndicator.tsx
@@ -20,7 +20,7 @@ const styles = (theme: Theme) => ({
  icon: {
    boxShadow: theme.shadows[2]
      .split('),')
-      .map((s) => `inset ${s}`)
+      .map(s => `inset ${s}`)
      .join('),'),
    padding: '6px',
    borderRadius: '50%',
--- a/app/src/reducers/Charts.ts
+++ b/app/src/reducers/Charts.ts
@@ -81,7 +81,7 @@ function addChart(state: ChartsState, action: AddChart) {

 function moveUp(state: ChartsState, action: MoveUp) {
  const charts = state.get('charts')
-  const idx = charts.findIndex((chart) => chart.topic === action.topic && chart.dotPath === action.dotPath)
+  const idx = charts.findIndex(chart => chart.topic === action.topic && chart.dotPath === action.dotPath)
  const item = charts.get(idx)
  const previousItem = charts.get(idx - 1)

@@ -95,7 +95,7 @@ function moveUp(state: ChartsState, action: MoveUp) {

 function updateChart(state: ChartsState, action: UpdateChart) {
  const charts = state.get('charts')
-  const chartIdx = charts.findIndex((chart) => chart.topic === action.topic && chart.dotPath === action.dotPath)
+  const chartIdx = charts.findIndex(chart => chart.topic === action.topic && chart.dotPath === action.dotPath)
  const chart = charts.get(chartIdx)

  return state.set('charts', chart ? charts.set(chartIdx, { ...chart, ...action.parameters }) : charts)
@@ -103,9 +103,7 @@ function updateChart(state: ChartsState, action: UpdateChart) {

 function removeChart(state: ChartsState, action: RemoveChart) {
  const charts = state.get('charts')
-  const newCharts = charts.filter(
-    (chart) => chart.topic !== action.chart.topic || chart.dotPath !== action.chart.dotPath
-  )
+  const newCharts = charts.filter(chart => chart.topic !== action.chart.topic || chart.dotPath !== action.chart.dotPath)
  return state.set('charts', newCharts)
 }

--- a/app/src/reducers/ConnectionManager.ts
+++ b/app/src/reducers/ConnectionManager.ts
@@ -160,7 +160,7 @@ function addSubscription(state: ConnectionManagerState, action: AddSubscription)

 function deleteSubscription(state: ConnectionManagerState, action: AddSubscription): ConnectionManagerState {
  const connection = state.connections[action.connectionId]
-  const newSubscriptions = connection.subscriptions.filter((s) => s !== action.subscription)
+  const newSubscriptions = connection.subscriptions.filter(s => s !== action.subscription)

  return {
    ...state,
--- a/app/src/reducers/Global.ts
+++ b/app/src/reducers/Global.ts
@@ -90,7 +90,7 @@ export const globalState: Reducer<Record<GlobalStateInterface>, GlobalAction> =
      }
      return state.set(
        'confirmationRequests',
-        state.get('confirmationRequests').filter((a) => a !== action.confirmationRequest)
+        state.get('confirmationRequests').filter(a => a !== action.confirmationRequest)
      )

    default:
--- a/app/src/sortedNodes.tsx
+++ b/app/src/sortedNodes.tsx
@@ -9,7 +9,7 @@ export function sortedNodes(settings: SettingsState, treeNode: q.TreeNode<any>):
  if (topicOrder === TopicOrder.abc) {
    edges.sort((a, b) => a.name.localeCompare(b.name))
  }
-  const nodes = edges.map((edge) => edge.target)
+  const nodes = edges.map(edge => edge.target)
  if (topicOrder === TopicOrder.messages) {
    nodes.sort((a, b) => b.leafMessageCount() - a.leafMessageCount())
  }