summaryrefslogtreecommitdiff
path: root/lib/Properties.hs
blob: 7d6fc4ae3df03345e8e95b9f96a9186a7f933eaf (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
{-# LANGUAGE NamedFieldPuns    #-}
{-# LANGUAGE OverloadedStrings #-}

-- | Contains checks for custom properties of the map json
module Properties (checkProperty) where


import           Control.Monad (unless)
import           Data.Text     (Text)
import           Tiled2        (Layer (layerProperties), Property, propertyName,
                                propertyValue)
import           Util          (prettyprint)

import           LintWriter    (LintWriter, complain, forbid, info, require,
                                suggest, warn)

-- | the point of this module
--
-- given a property, check if it is valid. It gets a reference
-- to its own layer since sometimes the presense of one property
-- implies the presence or absense of another.
--
-- The tests in here are meant to comply with the informal spec
-- at https://workadventu.re/map-building
--
-- I've attempted to build the LintWriter monad in a way
-- that should make this readable even to non-Haskellers
-- TODO: also pass the value of this property directly
checkProperty :: Layer -> Property -> LintWriter ()
checkProperty layer prop = case propName of
    "jitsiRoom" -> do
      info $ "found jitsi room: " <> prettyprint (propertyValue prop)
      suggestPropertyValue "jitsiTrigger" "onaction"
    "jitsiTrigger" ->
      requireProperty "jitsiRoom"
    "jitsiUrl" -> isForbidden
    "jitsiConfig" -> isForbidden
    "jitsiClientConfig" -> isForbidden
    "jitsiRoomAdminTag" -> isForbidden
    "playAudio" -> do
      -- TODO: check for url validity?
      pure ()
    "audioLoop" ->
      requireProperty "playAudio"
    "audioVolume" ->
      requireProperty "playAudio"
    "openWebsite" -> do
      suggestPropertyValue "openWebsiteTrigger" "onaction"
      require $ propertyValue prop
    "openWebsiteTrigger" ->
      requireProperty "openWebsite"
    "openWebsitePolicy" ->
      requireProperty "openWebsite"
    "exitUrl" -> pure ()
    "startLayer" -> pure ()
      -- could also make this a "hard error" (i.e. Left), but then it
      -- stops checking other properties as checkLayer short-circuits.
    _ -> warn $ "unknown property type " <> prettyprint propName
    where
      propName = propertyName prop
      -- | require some property in this layer
      requireProperty name = unless (hasProperty name layer)
        $ complain $ "property "<>prettyprint name<>" requires property "<>prettyprint propName
      -- | This property is forbidden and should not be used
      isForbidden = forbid $ "property " <> prettyprint propName <> " should not be used"
      -- TODO: check if the property has the correct value
      suggestPropertyValue :: Text -> Text -> LintWriter ()
      suggestPropertyValue name value = unless (hasProperty name layer)
        $ suggest $ "set property " <> prettyprint name <> " to " <> prettyprint value




-- | does this layer have the given property?
hasProperty :: Text -> Layer -> Bool
hasProperty name = any
  (\prop -> propertyName prop == name)
  . layerProperties