Skip to content

Commit f6eff52

Browse files
committed
fix(revert): revert bad change and refactor
1 parent 948aa93 commit f6eff52

2 files changed

Lines changed: 5 additions & 6 deletions

File tree

src/core/Flora/Import/Package.hs

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -77,6 +77,7 @@ import Distribution.Types.PackageDescription ()
7777
import Distribution.Types.TestSuite
7878
import Distribution.Types.Version (Version)
7979
import Distribution.Types.VersionRange (VersionRange, withinRange)
80+
import Distribution.Utils.ShortText (fromShortText)
8081
import Distribution.Version qualified as Version
8182
import Effectful
8283
import Effectful.Log (Log)
@@ -403,8 +404,9 @@ extractPackageDataFromCabal userId repository@(repositoryName, repositoryPackage
403404
let releaseId = deterministicReleaseId packageId packageVersion
404405
timestamp <- Time.currentTime
405406
let sourceRepos = getRepoURL packageName packageDesc.sourceRepos
406-
let categoryNames = map extractNameFromFloraCategories floraCategories
407-
let categories = Maybe.mapMaybe normaliseCategory categoryNames
407+
let rawCategoryField = packageDesc ^. #category % to fromShortText % to Text.pack
408+
let categoryList = fmap (Text.stripStart . Text.stripEnd) (Text.splitOn "," rawCategoryField)
409+
let categories = Maybe.mapMaybe normaliseCategory categoryList
408410
let package =
409411
Package
410412
{ packageId
@@ -482,9 +484,6 @@ extractPackageDataFromCabal userId repository@(repositoryName, repositoryPackage
482484
extractPackageDataFromCabal userId (repositoryName, repositoryPackages) uploadTime genericDesc
483485
Just components -> pure ImportOutput{..}
484486

485-
extractNameFromFloraCategories :: (Text, Text, Text) -> Text
486-
extractNameFromFloraCategories (_, name, _) = name
487-
488487
extractLibrary
489488
:: Package
490489
-> (Text, Set PackageName)

test/Flora/CategorySpec.hs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
module Flora.CategorySpec where
22

3-
import Test.Tasty
43
import Data.Set qualified as Set
4+
import Test.Tasty
55

66
import Flora.Normalise
77
import Flora.TestUtils

0 commit comments

Comments
 (0)