Skip to content

Commit

Permalink
Merge pull request Aspen-Discovery#1932 from Abhyastamita/24.08.00
Browse files Browse the repository at this point in the history
Sort volume numbers correctly
  • Loading branch information
mdnoble73 authored Jul 29, 2024
2 parents f822f98 + ced5b1e commit 1981fed
Show file tree
Hide file tree
Showing 4 changed files with 9 additions and 4 deletions.
3 changes: 3 additions & 0 deletions code/web/Drivers/CloudLibraryDriver.php
Original file line number Diff line number Diff line change
Expand Up @@ -943,6 +943,9 @@ public function redirectToCloudLibrary(User $patron, CloudLibraryRecordDriver $r
$postParams = [
'username' => $this->getPatronId($patron),
'password' => $this->getCloudLibraryPasswordOrPin($patron),
'eula' => 'eula',
'login_form' => 'true',
'library_id' => $settings->accountId,
];
$curlWrapper = new CurlWrapper();
$headers = [
Expand Down
2 changes: 2 additions & 0 deletions code/web/release_notes/24.08.00.MD
Original file line number Diff line number Diff line change
Expand Up @@ -75,9 +75,11 @@
// katherine
### Other Updates
- Fix bug with unexpected 404 errors on Web Builder pages. (Ticket 123122) (*KP*)
- Sort volume numbers correctly. (Ticket 133280) (*KP*)

### cloudLibrary Updates
- Add the ability to use alternate library cards (such as state library cards) with cloudLibrary. (Ticket 69336, 133101) (*KP*)
- Fix automatic login for Open in cloudLibrary button. (*KP*)

### Search Updates
- Fix bug where call number searches were not returning expected results. (Ticket 135530) (*KP*)
Expand Down
4 changes: 2 additions & 2 deletions code/web/sys/Grouping/Manifestation.php
Original file line number Diff line number Diff line change
Expand Up @@ -326,7 +326,7 @@ function getItemSummary() {
foreach ($this->_variations as $variation) {
$itemSummary = mergeItemSummary($itemSummary, $variation->getItemSummary());
}
ksort($itemSummary);
ksort($itemSummary, SORT_NATURAL);
$this->_itemSummary = $itemSummary;
$timer->logTime("Got item summary for manifestation");
}
Expand All @@ -345,7 +345,7 @@ function getItemsDisplayedByDefault() {
$itemsDisplayedByDefault = mergeItemSummary($itemsDisplayedByDefault, $variation->getItemsDisplayedByDefault());
}
}
ksort($itemsDisplayedByDefault);
ksort($itemsDisplayedByDefault, SORT_NATURAL);
$this->_itemsDisplayedByDefault = $itemsDisplayedByDefault;
}
return $this->_itemsDisplayedByDefault;
Expand Down
4 changes: 2 additions & 2 deletions code/web/sys/Grouping/Record.php
Original file line number Diff line number Diff line change
Expand Up @@ -470,7 +470,7 @@ public function addItemSummary($variationId, $key, $itemSummaryInfo, $groupedSta
}

public function sortItemSummary($variationId): void {
ksort($this->_itemSummary[$variationId]);
ksort($this->_itemSummary[$variationId], SORT_NATURAL);
}

/**
Expand Down Expand Up @@ -534,7 +534,7 @@ public function addItemDetails($variationId, $key, $itemSummaryInfo): void {
}

public function sortItemDetails($variationId): void {
ksort($this->_itemDetails[$variationId]);
ksort($this->_itemDetails[$variationId], SORT_NATURAL);
}

/**
Expand Down

0 comments on commit 1981fed

Please sign in to comment.