From eb10a2273636f1d1e7ed57babe2ec2f8f71188aa Mon Sep 17 00:00:00 2001 From: manuel-delvillar <68391066+manuel-delvillar@users.noreply.github.com> Date: Mon, 31 Jul 2023 10:35:48 -0600 Subject: [PATCH 1/2] Remove public url --- .../edu/wgu/osmt/api/model/ApiKeyword.kt | 19 +++++-------------- .../edu/wgu/osmt/keyword/KeywordController.kt | 18 ++++-------------- .../edu/wgu/osmt/security/SecurityConfig.kt | 1 - .../edu/wgu/osmt/api/model/ApiKeywordTest.kt | 4 +--- ...-manage-action-bar-vertical.component.html | 11 ----------- ...ta-manage-action-bar-vertical.component.ts | 10 ---------- .../named-reference/NamedReference.ts | 2 -- ui/test/resource/mock-data.ts | 3 +-- 8 files changed, 11 insertions(+), 57 deletions(-) diff --git a/api/src/main/kotlin/edu/wgu/osmt/api/model/ApiKeyword.kt b/api/src/main/kotlin/edu/wgu/osmt/api/model/ApiKeyword.kt index c9d25c1fc..ed2ea7b46 100644 --- a/api/src/main/kotlin/edu/wgu/osmt/api/model/ApiKeyword.kt +++ b/api/src/main/kotlin/edu/wgu/osmt/api/model/ApiKeyword.kt @@ -11,8 +11,7 @@ import org.springframework.beans.factory.annotation.Autowired @JsonInclude(JsonInclude.Include.ALWAYS) class ApiKeyword( private val keyword: Keyword, - private val totalSkills: Long?, - private val appConfig: AppConfig + private val totalSkills: Long? ) { @get:JsonProperty val id: Long? @@ -38,30 +37,22 @@ class ApiKeyword( val skillCount: Long? get() = totalSkills - @get:JsonProperty - val publicUrl: String - get() = "${appConfig.baseUrl}/api/metadata/keywords/${id}" - companion object { fun fromDao( - keywordDao: KeywordDao, - appConfig: AppConfig + keywordDao: KeywordDao ): ApiKeyword { return ApiKeyword( keyword = keywordDao.toModel(), - totalSkills = keywordDao.skills.count(), - appConfig = appConfig + totalSkills = keywordDao.skills.count() ) } fun fromModel( - keyword: Keyword, - appConfig: AppConfig + keyword: Keyword ): ApiKeyword { return ApiKeyword( totalSkills = keyword.id?.let { KeywordDao.findById(it)?.skills?.count() ?: 0 }, - keyword = keyword, - appConfig = appConfig + keyword = keyword ) } } diff --git a/api/src/main/kotlin/edu/wgu/osmt/keyword/KeywordController.kt b/api/src/main/kotlin/edu/wgu/osmt/keyword/KeywordController.kt index f611750ee..47026d66d 100644 --- a/api/src/main/kotlin/edu/wgu/osmt/keyword/KeywordController.kt +++ b/api/src/main/kotlin/edu/wgu/osmt/keyword/KeywordController.kt @@ -49,7 +49,6 @@ class KeywordController @Autowired constructor( val keywordEsRepo: KeywordEsRepo, val richSkillEsRepo: RichSkillEsRepo, val taskMessageService: TaskMessageService, - val appConfig: AppConfig, val oAuthHelper: OAuthHelper, ) { @@ -75,16 +74,7 @@ class KeywordController @Autowired constructor( return ResponseEntity.status(200) .headers(responseHeaders) - .body(searchResults.map { ApiKeyword.fromModel(it.content, appConfig) }.toList()) - } - - @RequestMapping(path = [ - "${RoutePaths.API}${RoutePaths.UNVERSIONED}${RoutePaths.KEYWORD_DETAIL}" - ], - produces = [MediaType.TEXT_HTML_VALUE]) - fun byUUIDHtmlView(@PathVariable id: String): String { - System.out.println("here by uuid html view keyword") - return "forward:${RoutePaths.UNVERSIONED}/metadata/keywords/$id" + .body(searchResults.map { ApiKeyword.fromModel(it.content) }.toList()) } @GetMapping( @@ -114,7 +104,7 @@ class KeywordController @Autowired constructor( return ResponseEntity .status(HttpStatus.OK) - .body(keywordRepository.createFromApi(apiKeywordUpdate)?.let { ApiKeyword(it.toModel(), it.skills.count(), appConfig) }) + .body(keywordRepository.createFromApi(apiKeywordUpdate)?.let { ApiKeyword(it.toModel(), it.skills.count()) }) } @PostMapping( @@ -136,7 +126,7 @@ class KeywordController @Autowired constructor( oAuthHelper.readableUserName(user) ) ?.let { - ApiKeyword(it.toModel(), it.skills.count(), appConfig) + ApiKeyword(it.toModel(), it.skills.count()) } ) } @@ -159,7 +149,7 @@ class KeywordController @Autowired constructor( private fun byId( id: Long, ): ApiKeyword? { - val found = keywordRepository.findById(id)?.let { ApiKeyword(it.toModel(), it.skills.count(), appConfig) } + val found = keywordRepository.findById(id)?.let { ApiKeyword(it.toModel(), it.skills.count()) } return found } diff --git a/api/src/main/kotlin/edu/wgu/osmt/security/SecurityConfig.kt b/api/src/main/kotlin/edu/wgu/osmt/security/SecurityConfig.kt index e47974934..8f65d2471 100644 --- a/api/src/main/kotlin/edu/wgu/osmt/security/SecurityConfig.kt +++ b/api/src/main/kotlin/edu/wgu/osmt/security/SecurityConfig.kt @@ -85,7 +85,6 @@ class SecurityConfig : WebSecurityConfigurerAdapter() { .mvcMatchers(GET, "${RoutePaths.API}${RoutePaths.API_V3}${RoutePaths.SKILL_DETAIL}", "${RoutePaths.API}${RoutePaths.API_V2}${RoutePaths.SKILL_DETAIL}", "${RoutePaths.API}${RoutePaths.UNVERSIONED}${RoutePaths.SKILL_DETAIL}").permitAll() - .mvcMatchers(GET,"${RoutePaths.API}${RoutePaths.UNVERSIONED}${RoutePaths.KEYWORD_DETAIL}").permitAll() .mvcMatchers(GET, "${RoutePaths.API}${RoutePaths.API_V3}${RoutePaths.COLLECTION_DETAIL}", "${RoutePaths.API}${RoutePaths.API_V2}${RoutePaths.COLLECTION_DETAIL}", "${RoutePaths.API}${RoutePaths.UNVERSIONED}${RoutePaths.COLLECTION_DETAIL}").permitAll() diff --git a/api/src/test/kotlin/edu/wgu/osmt/api/model/ApiKeywordTest.kt b/api/src/test/kotlin/edu/wgu/osmt/api/model/ApiKeywordTest.kt index 2fa9045d4..8ed0e21ee 100644 --- a/api/src/test/kotlin/edu/wgu/osmt/api/model/ApiKeywordTest.kt +++ b/api/src/test/kotlin/edu/wgu/osmt/api/model/ApiKeywordTest.kt @@ -1,13 +1,11 @@ package edu.wgu.osmt.api.model -import edu.wgu.osmt.config.AppConfig import edu.wgu.osmt.keyword.Keyword import edu.wgu.osmt.mockdata.MockData import org.assertj.core.api.Assertions import org.junit.jupiter.api.BeforeAll import org.junit.jupiter.api.Test import org.junit.jupiter.api.TestInstance -import org.springframework.beans.factory.annotation.Autowired @TestInstance(TestInstance.Lifecycle.PER_CLASS) internal class ApiKeywordTest { @@ -25,7 +23,7 @@ internal class ApiKeywordTest { val kw: Keyword = mockData.getKeywords().first() // Act - val api: ApiKeyword = ApiKeyword(kw, 7, mockData.appConfig) + val api: ApiKeyword = ApiKeyword(kw, 7) // Assert Assertions.assertThat(api.id).isEqualTo(kw.id) diff --git a/ui/src/app/metadata/detail/metadata-manage/action-bar-vertical/metadata-manage-action-bar-vertical.component.html b/ui/src/app/metadata/detail/metadata-manage/action-bar-vertical/metadata-manage-action-bar-vertical.component.html index 03af220a7..399db5323 100644 --- a/ui/src/app/metadata/detail/metadata-manage/action-bar-vertical/metadata-manage-action-bar-vertical.component.html +++ b/ui/src/app/metadata/detail/metadata-manage/action-bar-vertical/metadata-manage-action-bar-vertical.component.html @@ -10,17 +10,6 @@ Edit -