chore(medusa, utils): rename buildLegacyFieldsListFrom to objectToStringPath

.changeset/healthy-garlics-retire.md

@@ -0,0 +1,5 @@
+---
+"@medusajs/medusa": patch
+---
+
+chore(medusa): rename buildLegacyFieldsListFrom to objectToStringPath

packages/inventory/src/utils/query.ts

@@ -3,7 +3,7 @@ import {
   FilterableInventoryItemProps,
   FindConfig,
 } from "@medusajs/types"
-import { buildLegacyFieldsListFrom, buildQuery } from "@medusajs/utils"
+import { objectToStringPath, buildQuery } from "@medusajs/utils"
 import { EntityManager, FindOptionsWhere, ILike } from "typeorm"
 import { InventoryItem } from "../models"
 
@@ -60,7 +60,7 @@ export function getListQuery(
   }
 
   if (query.select) {
-    const legacySelect = buildLegacyFieldsListFrom(query.select)
+    const legacySelect = objectToStringPath(query.select)
     queryBuilder.select(legacySelect.map((s) => "inv_item." + s))
   }
 

packages/medusa/src/repositories/customer-group.ts

@@ -13,7 +13,7 @@ import {
   queryEntityWithIds,
   queryEntityWithoutRelations,
 } from "../utils/repository"
-import { buildLegacyFieldsListFrom } from "../utils"
+import { objectToStringPath } from "../utils"
 import { dataSource } from "../loaders/database"
 
 export type DefaultWithoutRelations = Omit<
@@ -135,10 +135,10 @@ export const CustomerGroupRepository = dataSource
         return [toReturn, toReturn.length]
       }
 
-      const legacyRelations = buildLegacyFieldsListFrom(relations)
+      const legacyRelations = objectToStringPath(relations)
       const groupedRelations = getGroupedRelations(legacyRelations)
 
-      const legacySelect = buildLegacyFieldsListFrom(
+      const legacySelect = objectToStringPath(
         idsOrOptionsWithoutRelations.select
       )
       const entitiesIdsWithRelations = await queryEntityWithIds(

packages/medusa/src/repositories/order.ts

@@ -1,7 +1,7 @@
 import { flatten, groupBy, map, merge } from "lodash"
 import { FindManyOptions, FindOptionsRelations, In } from "typeorm"
 import { Order } from "../models"
-import { buildLegacyFieldsListFrom } from "../utils"
+import { objectToStringPath } from "../utils"
 import { dataSource } from "../loaders/database"
 
 const ITEMS_REL_NAME = "items"
@@ -16,7 +16,7 @@ export const OrderRepository = dataSource.getRepository(Order).extend({
     const entitiesIds = entities.map(({ id }) => id)
 
     const groupedRelations: { [topLevel: string]: string[] } = {}
-    for (const rel of buildLegacyFieldsListFrom(relations)) {
+    for (const rel of objectToStringPath(relations)) {
       const [topLevel] = rel.split(".")
       if (groupedRelations[topLevel]) {
         groupedRelations[topLevel].push(rel)

packages/medusa/src/repositories/product-category.ts

@@ -9,7 +9,7 @@ import {
 import { ProductCategory } from "../models/product-category"
 import { ExtendedFindConfig, QuerySelector } from "../types/common"
 import { dataSource } from "../loaders/database"
-import { buildLegacyFieldsListFrom } from "../utils"
+import { objectToStringPath } from "../utils"
 import { isEmpty } from "lodash"
 
 export const ProductCategoryRepository = dataSource
@@ -44,8 +44,8 @@ export const ProductCategoryRepository = dataSource
       const options_ = { ...options }
       options_.where = options_.where as FindOptionsWhere<ProductCategory>
 
-      const legacySelect = buildLegacyFieldsListFrom(options_.select)
-      const legacyRelations = buildLegacyFieldsListFrom(options_.relations)
+      const legacySelect = objectToStringPath(options_.select)
+      const legacyRelations = objectToStringPath(options_.relations)
 
       const selectStatements = (relationName: string): string[] => {
         const modelColumns = this.metadata.ownColumns.map(

packages/medusa/src/repositories/product.ts

@@ -10,7 +10,7 @@ import { ExtendedFindConfig } from "../types/common"
 import { dataSource } from "../loaders/database"
 import { ProductFilterOptions } from "../types/product"
 import {
-  buildLegacyFieldsListFrom,
+  objectToStringPath,
   isObject,
   fetchCategoryDescendantsIds,
 } from "../utils"
@@ -89,7 +89,7 @@ export const ProductRepository = dataSource.getRepository(Product).extend({
     // https://github.com/typeorm/typeorm/issues/6294
     // Cleanup the repo and fix order/skip/take and relation load strategy when those issues are resolved
 
-    const orderFieldsCollectionPointSeparated = buildLegacyFieldsListFrom(
+    const orderFieldsCollectionPointSeparated = objectToStringPath(
       options.order ?? {}
     )
 

packages/medusa/src/repositories/tax-rate.ts

@@ -16,7 +16,7 @@ import {
 } from "../models"
 import { TaxRateListByConfig } from "../types/tax-rate"
 import { isDefined } from "medusa-core-utils"
-import { buildLegacyFieldsListFrom } from "../utils"
+import { objectToStringPath } from "../utils"
 import { dataSource } from "../loaders/database"
 
 const resolveableFields = [
@@ -33,7 +33,7 @@ export const TaxRateRepository = dataSource.getRepository(TaxRate).extend({
     const resolverFields: string[] = []
     if (isDefined(findOptions.select)) {
       const selectableCols: (keyof TaxRate)[] = []
-      const legacySelect = buildLegacyFieldsListFrom(
+      const legacySelect = objectToStringPath(
         findOptions.select as FindOptionsSelect<TaxRate>
       )
       for (const k of legacySelect) {

packages/medusa/src/utils/__tests__/build-query.spec.ts

@@ -1,5 +1,5 @@
 import { FindOptionsOrder, FindOptionsSelect, In, MoreThan, Not } from "typeorm"
-import { addOrderToSelect, buildLegacyFieldsListFrom, buildQuery } from "../build-query"
+import { addOrderToSelect, objectToStringPath, buildQuery } from "../build-query"
 
 describe("buildQuery", () => {
   it("successfully creates query", () => {
@@ -139,9 +139,9 @@ describe("buildQuery", () => {
   })
 })
 
-describe("buildLegacyFieldsListFrom", () => {
+describe("objectToStringPath", () => {
   it("successfully build back select object shape to list", () => {
-    const q = buildLegacyFieldsListFrom({
+    const q = objectToStringPath({
       order: {
         items: true,
         swaps: {
@@ -185,7 +185,7 @@ describe("buildLegacyFieldsListFrom", () => {
   })
 
   it("successfully build back relation object shape to list", () => {
-    const q = buildLegacyFieldsListFrom({
+    const q = objectToStringPath({
       order: {
         items: true,
         swaps: {
@@ -241,7 +241,7 @@ describe("buildLegacyFieldsListFrom", () => {
   })
 
   it("successfully build back order object shape to list", () => {
-    const q = buildLegacyFieldsListFrom({
+    const q = objectToStringPath({
       id: "ASC",
       items: {
         id: "ASC",

packages/medusa/src/utils/build-query.ts

@@ -153,7 +153,7 @@ function buildWhere<TWhereKeys extends object, TEntity>(
  * output: ['test.test1', 'test.test2', 'test.test3.test4', 'test2']
  * @param input
  */
-export function buildLegacyFieldsListFrom<TEntity>(
+export function objectToStringPath<TEntity>(
   input:
     | FindOptionsWhere<TEntity>
     | FindOptionsSelect<TEntity>
@@ -168,7 +168,7 @@ export function buildLegacyFieldsListFrom<TEntity>(
 
   for (const key of Object.keys(input)) {
     if (input[key] != undefined && typeof input[key] === "object") {
-      const deepRes = buildLegacyFieldsListFrom(input[key])
+      const deepRes = objectToStringPath(input[key])
 
       const items = deepRes.reduce((acc, val) => {
         acc.push(`${key}.${val}`)

packages/utils/src/common/build-query.ts

@@ -153,7 +153,7 @@ function buildWhere<TWhereKeys extends object, TEntity>(
  * output: ['test.test1', 'test.test2', 'test.test3.test4', 'test2']
  * @param input
  */
-export function buildLegacyFieldsListFrom<TEntity>(
+export function objectToStringPath<TEntity>(
   input:
     | FindOptionsWhere<TEntity>
     | FindOptionsSelect<TEntity>
@@ -168,7 +168,7 @@ export function buildLegacyFieldsListFrom<TEntity>(
 
   for (const key of Object.keys(input)) {
     if (input[key] != undefined && typeof input[key] === "object") {
-      const deepRes = buildLegacyFieldsListFrom(input[key])
+      const deepRes = objectToStringPath(input[key])
 
       const items = deepRes.reduce((acc, val) => {
         acc.push(`${key}.${val}`)