From b09b7513a723e7e8b0c0cd24d5fa91bb946d6895 Mon Sep 17 00:00:00 2001
From: Harpo Harbert <ryan_harbert@wgbh.org>
Date: Thu, 12 Dec 2024 15:00:04 -0800
Subject: [PATCH] Fixes accessLevel not defined error

---
 app/classes/aapbRecordHelpers.tsx | 84 +++++++++++++++----------------
 app/types/aapb.tsx                |  1 +
 2 files changed, 41 insertions(+), 44 deletions(-)

diff --git a/app/classes/aapbRecordHelpers.tsx b/app/classes/aapbRecordHelpers.tsx
index 403737a..e941461 100644
--- a/app/classes/aapbRecordHelpers.tsx
+++ b/app/classes/aapbRecordHelpers.tsx
@@ -11,17 +11,13 @@ export function handleAapbRecordGroup(aapbRecordGroup, key) {
   // this func is where we split by whitespace v
   var guids = parseAapbRecordGroup(aapbRecordGroup.value.guids)
 
-  // preserve these flags' effect for each aapb_record_group
-  var showThumbnail = aapbRecordGroup.value.show_thumbnail
-  var showTitle = aapbRecordGroup.value.show_title
-
   return (
     <AAPBRecords
       guids={guids}
       startTime={aapbRecordGroup.value.start_time}
       endTime={aapbRecordGroup.value.end_time}
-      showThumbnail={showThumbnail}
-      showTitle={showTitle}
+      showThumbnail={aapbRecordGroup.value.show_thumbnail}
+      showTitle={aapbRecordGroup.value.show_title}
       accessLevel={aapbRecordGroup.value.access_level}
       embedPlayer={true}
       specialCollections={
@@ -74,16 +70,16 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
         inst = [inst]
       }
       // Find all proxies
-      let proxies = inst.find(i => i.instantiationGenerations == 'Proxy')
+      let proxies = inst.find((i) => i.instantiationGenerations == 'Proxy')
       if (proxies) {
         // proxytome proxytome proxytome proxytome
         if (!(proxies instanceof Array)) {
           proxies = [proxies]
         }
         // Get the aspect ratio of the essence tracks
-        let ets = proxies.map(proxy =>
+        let ets = proxies.map((proxy) =>
           proxy.instantiationEssenceTrack.map(
-            track => track.essenceTrackAspectRatio
+            (track) => track.essenceTrackAspectRatio
           )
         )
         for (let aspect of ets) {
@@ -97,9 +93,13 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
         }
       }
     }
-  
-    this.setState({ guid: hyphenGuid, pbcore: record, wide: isWide, mediaType: this.mediaType(record) })
 
+    this.setState({
+      guid: hyphenGuid,
+      pbcore: record,
+      wide: isWide,
+      mediaType: this.mediaType(record),
+    })
   }
 
   mediaType(pbcore: PBCore) {
@@ -166,7 +166,7 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
         <iframe
           className={iframeClasses}
           src={url}
-          frameBorder="0"
+          frameBorder='0'
           allowFullScreen={true}
         />
       </a>
@@ -180,9 +180,9 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
       let titleBar
       if (this.props.showTitle) {
         titleBar = (
-          <div className="shade-bar">
+          <div className='shade-bar'>
             <a href={`https://americanarchive.org/catalog/${this.state.guid}`}>
-              { this.aapbTitle(this.state.pbcore) }
+              {this.aapbTitle(this.state.pbcore)}
             </a>
           </div>
         )
@@ -191,11 +191,11 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
       let thumbnail
       if (this.props.showThumbnail) {
         let mt = this.state.mediaType
-        if (mt == "Moving Image") {
+        if (mt == 'Moving Image') {
           // check here for digitized? if not show VIDEO THUMB
           var ci_pbi =
             this.state.pbcore.pbcoreDescriptionDocument.pbcoreIdentifier.find(
-              pbi => pbi.source == 'Sony Ci'
+              (pbi) => pbi.source == 'Sony Ci'
             )
           if (ci_pbi && ci_pbi.text) {
             thumbnail = {
@@ -207,7 +207,7 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
               backgroundImage: `url(/VIDEO_SMALL.png)`,
             }
           }
-        } else if(mt == "Sound") {
+        } else if (mt == 'Sound') {
           // AUDIO THUMB
           thumbnail = {
             backgroundImage: `url(/AUDIO_SMALL.png)`,
@@ -220,9 +220,9 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
       }
 
       let playButton
-      if(this.state.mediaType){
+      if (this.state.mediaType) {
         playButton = (
-          <div className="blue-circle">
+          <div className='blue-circle'>
             <div />
           </div>
         )
@@ -235,16 +235,14 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
           this.state.wide
         )
       } else {
-
         if (this.props.embedPlayer) {
           recordBlock = (
             <a
               style={thumbnail}
-              className="content-aapbblock"
-              onClick={() => this.setState({ showEmbed: true })}
-            >
-              { titleBar }
-              { playButton }
+              className='content-aapbblock'
+              onClick={() => this.setState({ showEmbed: true })}>
+              {titleBar}
+              {playButton}
             </a>
           )
         } else {
@@ -252,11 +250,10 @@ export class AAPBRecord extends Component<AAPBRecordProps> {
           recordBlock = (
             <a
               style={thumbnail}
-              className="content-aapbblock"
-              href={this.aapbCatalogURL(this.state.guid)}
-            >
-              { titleBar }
-              { playButton }
+              className='content-aapbblock'
+              href={this.aapbCatalogURL(this.state.guid)}>
+              {titleBar}
+              {playButton}
             </a>
           )
         }
@@ -279,21 +276,16 @@ export class AAPBRecords extends Component<AAPBRecordBlockProps> {
   }
 
   async componentDidMount() {
-    var accessLevel = "online"
-    if(this.props.accessLevel){
-      accessLevel = this.props.accessLevel
-    }
-    
     this.setState({ aapb_host: window.ENV.AAPB_HOST }, async () => {
       var data
       if (this.props.specialCollections) {
         // fetch actual number of records from this special collection search
         data = await fetch(
-        // this endpoint takes a bare solr query within each filter option (q, fq, etc.), NOT BLACKLIGHT URL PARAMS
-        `${window.ENV.AAPB_HOST}/api.json?fq=special_collections:${this.props.specialCollections} AND access_types:${accessLevel}&sort=title+asc&rows=0`
+          // this endpoint takes a bare solr query within each filter option (q, fq, etc.), NOT BLACKLIGHT URL PARAMS
+          `${window.ENV.AAPB_HOST}/api.json?fq=special_collections:${this.props.specialCollections} AND access_types:${this.props.accessLevel}&sort=title+asc&rows=0`
         )
-        .then(response => response.json())
-        .catch(error => console.error(error))
+          .then((response) => response.json())
+          .catch((error) => console.error(error))
       }
       if (data) {
         this.setState({ numRecords: parseInt(data['response']['numFound']) })
@@ -312,6 +304,7 @@ export class AAPBRecords extends Component<AAPBRecordBlockProps> {
           showTitle={this.state.showTitle}
           startTime={this.props.startTime}
           endTime={this.props.endTime}
+          accessLevel={this.props.accessLevel}
           specialCollections={this.props.specialCollections}
         />
       )
@@ -319,11 +312,11 @@ export class AAPBRecords extends Component<AAPBRecordBlockProps> {
 
     var recordsSearchLink = `${this.state.aapb_host}/catalog`
     if (this.props.specialCollections) {
-      recordsSearchLink += `?f[special_collections][]=${this.props.specialCollections}&sort=title+asc&f[access_types][]=${accessLevel}`
+      recordsSearchLink += `?f[special_collections][]=${this.props.specialCollections}&sort=title+asc&f[access_types][]=${this.props.accessLevel}`
     }
     var msg
-    if(this.state.numRecords > 0){
-      if(this.state.numRecords == 1){
+    if (this.state.numRecords > 0) {
+      if (this.state.numRecords == 1) {
         msg = `View on AAPB >`
       } else {
         msg = `View all ${this.state.numRecords} on AAPB >`
@@ -332,9 +325,12 @@ export class AAPBRecords extends Component<AAPBRecordBlockProps> {
       msg = `View more on AAPB >`
     }
     return (
-      <div className="aapb-records">
+      <div className='aapb-records'>
         {aapbRecords}
-        <a target="_blank" className="aapb-records-seemore" href={recordsSearchLink}>
+        <a
+          target='_blank'
+          className='aapb-records-seemore'
+          href={recordsSearchLink}>
           {msg}
         </a>
       </div>
diff --git a/app/types/aapb.tsx b/app/types/aapb.tsx
index 9d6ccfc..9e29b04 100644
--- a/app/types/aapb.tsx
+++ b/app/types/aapb.tsx
@@ -7,6 +7,7 @@ export type AAPBRecordProps = {
   embedPlayer: boolean
   startTime: string
   endTime: string
+  accessLevel: string
 }
 
 export type AAPBRecordState = {