Needs review
Project:
Panopoly
Version:
7.x-1.x-dev
Component:
Widgets
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Issue tags:
Reporter:
Created:
17 Apr 2015 at 20:06 UTC
Updated:
8 May 2018 at 12:37 UTC
Jump to comment: Most recent, Most recent file
Comments
Comment #1
mpotter commentedPatch to panopoly_widgets is here.
Comment #2
mpotter commentedDerp, typo fail!
Comment #3
mpotter commentedWell shoot, this gets annoying with multiple patches. OA already applies a patch to panopoly_widgets for [media], so we need to combine them for this to work in OA. So here is a consolidated patch including #2458039: Allow multi select for media items.
This applies to the -dev version of panopoly_widgets, not the 1.20 release.
Comment #4
mpotter commentedPatch updated for latest panopoly_widgets dev
Comment #5
mpotter commentedComment #6
mpotter commentedComment #7
mpotter commentedLooks like there was a further issue for Media Browser not loading it's javascript.
This new patch contains the updated list of Media patches for Panopoly used in Open Atrium:
#951004: Allow selecting of multiple media items for a multi value media field in the same dialog
#2473481: Clicking images in media browser doesn't select them
#2528428: Javascript not loaded when Media Browser view is empty
These should be added to Panopoly when they update the Media module to the latest version.
Comment #8
mpotter commentedHere is the updated patch for latest Panopoly dev
Comment #9
mpotter commentedHere is a better filename for the patch to mirror what we do in panopoly_core
Comment #10
mpotter commentedRenaming this to something closer to what it's being used for now.
Comment #11
mpotter commentedAdding the patch from #2534724: Browser window opens twice for field image until #2618206: Patching media for opening multiple media browser popups is committed.
Comment #12
mpotter commentedThat was fast. #2534724: Browser window opens twice for field image is committed, so here is the updated patch
Comment #13
mpotter commentedOK, and there is a commit from the media -dev branch that we also need here for multi-select. Documented in #2549809: Status of "Fixed media browser appearing once for each time a field is repeated on the same page."
Comment #14
mpotter commentedSigh...forgot to re-roll that against latest panopoly_widget dev. Here it is again.
Comment #15
mpotter commentedUpdating patch for latest dev
Comment #16
mpotter commentedAnd another fix
Comment #17
mpotter commentedUpdating patch to latest dev.
Comment #18
mpotter commentedHere is a temp patch that also updates Media to 2.0 pending this getting fixed in Panopoly Widgets directly.
Comment #19
mpotter commentedSigh...here is a version that applies to Panopoly 1.43 instead of dev (#18 is for dev)
Comment #20
mpotter commentedThis is for the latest dev
Comment #21
mpotter commentedRe-roll for latest dev
Comment #22
joseph.olstadsee #2931757: upgrade media and media_youtube in panopoly_widgets
Comment #23
joseph.olstadthis patch to panopoly_widgets #2931757-3: upgrade media and media_youtube in panopoly_widgets
Comment #24
joseph.olstadPatch needed for file_entity
Please see:
#2932229-14: Unable to create or edit a Document Page in Section unless administrator
Here is the new panopoly_widgets patch.
Comment #25
joseph.olstadPatch 24 above resolves a critical (imho) regression with the latest releases of openatrium.
Comment #26
mpotter commentedPlease ignore the patch posted in #24. This issue thread is only for Media patches to Atrium/Panopoly. The patches for File Entity are in #2477397: Patches for file_entity from OpenAtrium.
The patch in #21 is still what Atrium is using for now.
Comment #27
mpotter commentedUpdated patch for just updating Media to 2.19 so Atrium doesn't need to wait on Panopoly release
Comment #28
mpotter commentedHere is the next version to apply against Panopoly 1.53 to bring Media up to 2.19.
Comment #29
mpotter commentedLatest panopoly dev updates to media 2.19 so this patch no longer needed in Atrium.