fix some of ill reservation problems, where 'all' shold be used inste…

…ad of 'mostRelevant'

src/components/_modal/pages/order/localizationinformation/LocalizationInformation.js

@@ -88,6 +88,7 @@ export function LocalizationInformation({
  >{`${pickupBranch?.postalCode} ${pickupBranch?.city}`}</Text>
  </div>
  )}
+
  {/* maybe move warning together with warning in order-modal. see bibdk2021-1927 */}
  {!isLoadingBranches &&
  pickupBranch &&

src/components/hooks/useOrderPageInformations.js

@@ -40,7 +40,7 @@ export default function useOrderPageInformation({
  return inferAccessTypes(
  periodicaForm,
  pickupBranch,
- workData?.work?.manifestations?.mostRelevant,
+ workData?.work?.manifestations?.all,
  loanerInfo
  );
  }, [workData?.work, periodicaForm, pickupBranch]);

src/components/hooks/useWorkAndSelectedPids.js

@@ -13,14 +13,10 @@ export function useGetManifestationsForOrderButton(workId, selectedPids) {
  const workResponse = useData(workId && workFragments.buttonTxt({ workId }));
 
  const allPids = useMemo(() => {
- return workResponse?.data?.work?.manifestations?.mostRelevant?.flatMap(
+ return workResponse?.data?.work?.manifestations?.all?.flatMap(
  (manifestation) => manifestation.pid
  );
- }, [
- workId,
- selectedPids,
- workResponse?.data?.work?.manifestations?.mostRelevant,
- ]);
+ }, [workId, selectedPids, workResponse?.data?.work?.manifestations?.all]);
 
  const manifestationsResponse = useData(
  allPids &&

src/components/work/overview/Overview.js

@@ -63,7 +63,7 @@ export function Overview({
  className = "",
  skeleton = false,
 }) {
- const manifestations = work?.manifestations?.mostRelevant;
+ const manifestations = work?.manifestations?.all;
  const router = useRouter();
 
  const { uniqueMaterialTypes, inUniqueMaterialTypes, flatPidsByType } =

src/components/work/reservationbutton/ReservationButton.js

@@ -158,7 +158,7 @@ export const ReservationButton = ({
  modal,
  handleOrderFinished = undefined,
 }) => {
- const physicalCopy = checkPhysicalCopy([access?.[0]])?.[0]; //TODO why do we check all accesses if only one is used in the end?
+ const physicalCopy = checkPhysicalCopy(access)?.[0]; //TODO why do we check all accesses if only one is used in the end?
  const digitalCopy = checkDigitalCopy([access?.[0]])?.[0]; //TODO why do we check all accesses if only one is used in the end?
 
  const noSelectedManifestations = Boolean(isEmpty(access));

src/components/work/reservationbutton/orderbuttontextbelow/OrderButtonTextBelow.js

@@ -24,7 +24,7 @@ import useUser from "@/components/hooks/useUser";
  * @returns {React.ReactElement|null}
  */
 function OrderButtonTextBelow({ access, skeleton }) {
- const physicalCopy = checkPhysicalCopy([access?.[0]])?.[0];
+ const physicalCopy = checkPhysicalCopy(access)?.[0];
  const digitalCopy = checkDigitalCopy([access?.[0]])?.[0];
  const isPeriodicaLike = getAreAccessesPeriodicaLike([access?.[0]])?.[0];
 

src/lib/api/work.fragments.js

@@ -438,6 +438,9 @@ export function buttonTxt({ workId }) {
  manifestations {
  all {
  pid
+ materialTypes {
+ ...materialTypesFragment
+ }
  }
  mostRelevant {
  pid
@@ -766,7 +769,7 @@ export function orderPageWorkWithManifestations({ workId }) {
  }
  workTypes
  manifestations {
- mostRelevant {
+ all {
  ...manifestationAccess
  ...manifestationDetailsForAccessFactory
  }
@@ -807,7 +810,7 @@ export function overviewWork({ workId }) {
  }
  workTypes
  manifestations {
- mostRelevant {
+ all { 
  ownerWork {
  workTypes
  }