From: <die...@us...> - 2010-06-29 15:28:29
|
Revision: 2727 http://openutils.svn.sourceforge.net/openutils/?rev=2727&view=rev Author: diego_schivo Date: 2010-06-29 15:28:18 +0000 (Tue, 29 Jun 2010) Log Message: ----------- MEDIA-145 number of items per media-type retrieved using Magnolia Criteria API Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-29 15:19:46 UTC (rev 2726) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-29 15:28:18 UTC (rev 2727) @@ -173,18 +173,15 @@ numberOfMedia = new HashMap<String, Integer>(); for (MediaTypeConfiguration mtc : types) { - try - { - Collection<Content> c = MediaConfigurationManager.getInstance().find( - path, - mtc.getName(), - search, - StringUtils.isNotBlank(search)); - numberOfMedia.put(mtc.getName(), c.size()); - } - catch (RepositoryException e) - { - } + AdvancedResult typeResult = MediaConfigurationManager.getInstance().find( + path, + mtc.getName(), + search, + StringUtils.isEmpty(search), + null, + 0, + -1); + numberOfMedia.put(mtc.getName(), (int) typeResult.getItems().getSize()); } } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 06:24:47
|
Revision: 2734 http://openutils.svn.sourceforge.net/openutils/?rev=2734&view=rev Author: diego_schivo Date: 2010-06-30 06:24:40 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 refactoring Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 06:17:44 UTC (rev 2733) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 06:24:40 UTC (rev 2734) @@ -40,6 +40,7 @@ import java.io.IOException; import java.util.ArrayList; import java.util.Collection; +import java.util.Collections; import java.util.HashMap; import java.util.Iterator; import java.util.List; @@ -230,7 +231,6 @@ @Override public String show() { - HierarchyManager hm = MgnlContext.getInstance().getHierarchyManager(MediaModule.REPO); final MediaTypeConfiguration mtc = MediaConfigurationManager.getInstance().getTypes().get(type); AdvancedResult typeResult = null; @@ -251,7 +251,6 @@ sorting, page - 1, 5); - pages = MgnlPagingElFunctions.pageList(typeResult.getNumberOfPages(), 5, "page"); } } else if (!StringUtils.isBlank(search)) @@ -262,16 +261,14 @@ typeResult = MediaConfigurationManager.getInstance().find(null, type, search, false, sorting, 0, -1); } - medias = null; + medias = CollectionUtils.EMPTY_COLLECTION; + pages = Collections.emptyList(); if (typeResult != null) { ResultIterator<AdvancedResultItem> items = typeResult.getItems(); medias = Lists.newArrayList(Iterators.transform(items, new MediaBeanBuilder(mtc))); + pages = MgnlPagingElFunctions.pageList(typeResult.getNumberOfPages(), 5, "page"); } - if (medias == null) - { - medias = CollectionUtils.EMPTY_COLLECTION; - } return super.show(); } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 08:59:44
|
Revision: 2744 http://openutils.svn.sourceforge.net/openutils/?rev=2744&view=rev Author: diego_schivo Date: 2010-06-30 08:59:38 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 moved call to fillNumberOfMediaPerType() from init() to show() because sub-classes (adv-search pages) do not need it Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 08:36:11 UTC (rev 2743) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 08:59:38 UTC (rev 2744) @@ -177,11 +177,6 @@ types = MediaConfigurationManager.getInstance().getTypes().values(); } - if (!StringUtils.isBlank(path) || !StringUtils.isBlank(search)) - { - fillNumberOfMediaPerType(); - } - bgSelector = null; sorting = null; for (Cookie cookie : request.getCookies()) @@ -228,7 +223,10 @@ @Override public String show() { - final MediaTypeConfiguration mtc = MediaConfigurationManager.getInstance().getTypes().get(type); + if (!StringUtils.isBlank(path) || !StringUtils.isBlank(search)) + { + fillNumberOfMediaPerType(); + } AdvancedResult typeResult = null; @@ -262,8 +260,8 @@ pages = Collections.emptyList(); if (typeResult != null) { - ResultIterator<AdvancedResultItem> items = typeResult.getItems(); - medias = Iterators.transform(items, new MediaBeanBuilder(mtc)); + MediaTypeConfiguration mtc = MediaConfigurationManager.getInstance().getTypes().get(type); + medias = Iterators.transform(typeResult.getItems(), new MediaBeanBuilder(mtc)); pages = MgnlPagingElFunctions.pageList(typeResult.getNumberOfPages(), 5, "page"); } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 09:47:14
|
Revision: 2747 http://openutils.svn.sourceforge.net/openutils/?rev=2747&view=rev Author: diego_schivo Date: 2010-06-30 09:47:08 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 refactoring Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 09:28:32 UTC (rev 2746) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 09:47:08 UTC (rev 2747) @@ -53,8 +53,6 @@ import javax.servlet.http.HttpServletResponse; import net.sourceforge.openutils.mgnlcriteria.jcr.query.AdvancedResult; -import net.sourceforge.openutils.mgnlcriteria.jcr.query.AdvancedResultItem; -import net.sourceforge.openutils.mgnlcriteria.jcr.query.ResultIterator; import net.sourceforge.openutils.mgnlmedia.media.configuration.MediaConfigurationManager; import net.sourceforge.openutils.mgnlmedia.media.configuration.MediaTypeConfiguration; import net.sourceforge.openutils.mgnlmedia.media.lifecycle.MediaModule; @@ -157,6 +155,7 @@ if (!StringUtils.isEmpty(mediaType)) { + // subset of types specified by selectMedia control String[] mediaTypes = StringUtils.split(mediaType, ","); type = mediaTypes[0]; types = new ArrayList<MediaTypeConfiguration>(); @@ -165,13 +164,12 @@ types.add(MediaConfigurationManager.getInstance().getTypes().get(mt)); } } - else if (type == null) - { - type = MediaConfigurationManager.getInstance().getTypes().entrySet().iterator().next().getKey(); - types = MediaConfigurationManager.getInstance().getTypes().values(); - } else { + if (StringUtils.isEmpty(type)) + { + type = MediaConfigurationManager.getInstance().getTypes().keySet().iterator().next(); + } types = MediaConfigurationManager.getInstance().getTypes().values(); } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 10:09:43
|
Revision: 2748 http://openutils.svn.sourceforge.net/openutils/?rev=2748&view=rev Author: diego_schivo Date: 2010-06-30 10:09:36 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 refactoring Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 09:47:08 UTC (rev 2747) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:09:36 UTC (rev 2748) @@ -153,6 +153,7 @@ { super.init(); + // left side of navigation bar if (!StringUtils.isEmpty(mediaType)) { // subset of types specified by selectMedia control @@ -173,6 +174,7 @@ types = MediaConfigurationManager.getInstance().getTypes().values(); } + // right side of navigation bar bgSelector = null; sorting = null; for (Cookie cookie : request.getCookies()) @@ -189,51 +191,48 @@ bgSelector = StringUtils.defaultIfEmpty(bgSelector, "white"); sorting = StringUtils.defaultIfEmpty(sorting, SORT_BY_CREATIONDATE); + // for activation status handling develop = SystemProperty.getBooleanProperty("magnolia.develop"); } /** - * Count medias for each media type + * Counts media items for the specified type. + * @param mtc + * @return */ - private void fillNumberOfMediaPerType() + protected int countMediaItems(MediaTypeConfiguration mtc) { - numberOfMedia = new HashMap<String, Integer>(); - for (MediaTypeConfiguration mtc : types) + if (StringUtils.isEmpty(path) && StringUtils.isBlank(search)) { - AdvancedResult typeResult = MediaConfigurationManager.getInstance().find( - path, - mtc.getName(), - search, - StringUtils.isEmpty(search), - null, - 0, - 1); - numberOfMedia.put(mtc.getName(), (int) typeResult.getItems().getSize()); + return 0; } + numberOfMedia = new HashMap<String, Integer>(); + AdvancedResult typeResult = MediaConfigurationManager.getInstance().find( + path, + mtc.getName(), + search, + StringUtils.isEmpty(search), + null, + 0, + 1); + return (int) typeResult.getItems().getSize(); } /** - * {@inheritDoc} + * Finds media items to show. + * @return */ - @Override - public String show() + protected AdvancedResult findMediaItems() { - if (!StringUtils.isBlank(path) || !StringUtils.isBlank(search)) + AdvancedResult result = null; + if (!StringUtils.isEmpty(path)) { - fillNumberOfMediaPerType(); - } - - AdvancedResult typeResult = null; - - if (!StringUtils.isBlank(path)) - { Content folder = ContentUtil.getContent(MediaModule.REPO, path); if (folder != null) { writable = folder.isGranted(Permission.WRITE); canPublish = writable && ActivationManagerFactory.getActivationManager().hasAnyActiveSubscriber(); - - typeResult = MediaConfigurationManager.getInstance().find( + result = MediaConfigurationManager.getInstance().find( folder.getHandle(), type, null, @@ -247,19 +246,33 @@ { writable = false; canPublish = false; + result = MediaConfigurationManager.getInstance().find(null, type, search, false, sorting, 0, 1); + } + return result; + } - typeResult = MediaConfigurationManager.getInstance().find(null, type, search, false, sorting, 0, -1); + /** + * {@inheritDoc} + */ + @Override + public String show() + { + numberOfMedia = new HashMap<String, Integer>(); + for (MediaTypeConfiguration mtc : types) + { + numberOfMedia.put(mtc.getName(), countMediaItems(mtc)); } medias = Iterators.emptyIterator(); pages = Collections.emptyList(); - if (typeResult != null) + AdvancedResult result = findMediaItems(); + if (result != null) { MediaTypeConfiguration mtc = MediaConfigurationManager.getInstance().getTypes().get(type); // casts Iterator<AdvancedResultItem> to Iterator<Content> - Iterator<Content> contentIterator = Iterators.filter(typeResult.getItems(), Content.class); + Iterator<Content> contentIterator = Iterators.filter(result.getItems(), Content.class); medias = Iterators.transform(contentIterator, new MediaBeanBuilder(mtc)); - pages = MgnlPagingElFunctions.pageList(typeResult.getNumberOfPages(), 5, "page"); + pages = MgnlPagingElFunctions.pageList(result.getNumberOfPages(), 5, "page"); } return super.show(); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 10:14:16
|
Revision: 2749 http://openutils.svn.sourceforge.net/openutils/?rev=2749&view=rev Author: diego_schivo Date: 2010-06-30 10:14:10 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 fix Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:09:36 UTC (rev 2748) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:14:10 UTC (rev 2749) @@ -206,7 +206,6 @@ { return 0; } - numberOfMedia = new HashMap<String, Integer>(); AdvancedResult typeResult = MediaConfigurationManager.getInstance().find( path, mtc.getName(), This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 10:37:06
|
Revision: 2754 http://openutils.svn.sourceforge.net/openutils/?rev=2754&view=rev Author: diego_schivo Date: 2010-06-30 10:37:00 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 total-size (avoid getting iterator) Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:32:48 UTC (rev 2753) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:37:00 UTC (rev 2754) @@ -214,7 +214,7 @@ null, 0, 1); - return (int) typeResult.getItems().getSize(); + return typeResult.getTotalSize(); } /** This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-06-30 10:51:14
|
Revision: 2755 http://openutils.svn.sourceforge.net/openutils/?rev=2755&view=rev Author: diego_schivo Date: 2010-06-30 10:51:08 +0000 (Wed, 30 Jun 2010) Log Message: ----------- MEDIA-145 sorting must be specified for total-size to work (-1 otherwise) Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:37:00 UTC (rev 2754) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-06-30 10:51:08 UTC (rev 2755) @@ -211,7 +211,8 @@ path, search, StringUtils.isEmpty(search), - null, + // sorting must be specified for total-size to work (-1 otherwise) + SORT_BY_CREATIONDATE, 0, 1); return typeResult.getTotalSize(); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <die...@us...> - 2010-09-10 14:55:44
|
Revision: 3030 http://openutils.svn.sourceforge.net/openutils/?rev=3030&view=rev Author: diego_schivo Date: 2010-09-10 14:55:38 +0000 (Fri, 10 Sep 2010) Log Message: ----------- MEDIA-177 fix Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-09-10 14:46:23 UTC (rev 3029) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-09-10 14:55:38 UTC (rev 3030) @@ -259,7 +259,7 @@ { itemsPerPage = module.getFolderViewItemsPerPage(); } - return module.getSearch().search(request, type, path, false, sorting, module.getFolderViewItemsPerPage(), page); + return module.getSearch().search(request, type, path, false, sorting, itemsPerPage, page); } /** This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <fg...@us...> - 2010-09-10 15:33:01
|
Revision: 3035 http://openutils.svn.sourceforge.net/openutils/?rev=3035&view=rev Author: fgiust Date: 2010-09-10 15:32:55 +0000 (Fri, 10 Sep 2010) Log Message: ----------- MEDIA-179 handle default sorting Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-09-10 15:29:16 UTC (rev 3034) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-09-10 15:32:55 UTC (rev 3035) @@ -172,7 +172,7 @@ } bgSelector = StringUtils.defaultIfEmpty(bgSelector, "white"); pagesizeSelector = StringUtils.defaultIfEmpty(pagesizeSelector, "size1"); - if (SortMode.valueOf(sorting) == null) + if (sorting == null || SortMode.valueOf(sorting) == null) { sorting = getDefaultSorting().name(); } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <fg...@us...> - 2010-09-10 16:02:49
|
Revision: 3037 http://openutils.svn.sourceforge.net/openutils/?rev=3037&view=rev Author: fgiust Date: 2010-09-10 16:02:43 +0000 (Fri, 10 Sep 2010) Log Message: ----------- MEDIA-179 don't crash on invalid sort parameters Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-09-10 15:42:15 UTC (rev 3036) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2010-09-10 16:02:43 UTC (rev 3037) @@ -172,8 +172,16 @@ } bgSelector = StringUtils.defaultIfEmpty(bgSelector, "white"); pagesizeSelector = StringUtils.defaultIfEmpty(pagesizeSelector, "size1"); - if (sorting == null || SortMode.valueOf(sorting) == null) + + try { + if (sorting == null || SortMode.valueOf(sorting) == null) + { + sorting = getDefaultSorting().name(); + } + } + catch (IllegalArgumentException e) + { sorting = getDefaultSorting().name(); } // for activation status handling This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <fg...@us...> - 2011-11-06 17:12:49
|
Revision: 3686 http://openutils.svn.sourceforge.net/openutils/?rev=3686&view=rev Author: fgiust Date: 2011-11-06 17:12:42 +0000 (Sun, 06 Nov 2011) Log Message: ----------- use media/activate as command name also in the folder view Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2011-11-06 17:09:49 UTC (rev 3685) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2011-11-06 17:12:42 UTC (rev 3686) @@ -430,7 +430,7 @@ @SuppressWarnings("unchecked") public String activate() { - Command cmd = CommandsManager.getInstance().getCommand(CommandsManager.DEFAULT_CATALOG, "activateMedia"); + Command cmd = CommandsManager.getInstance().getCommand("media", "activate"); ActivationCommand actCmd = (ActivationCommand) cmd; @@ -466,7 +466,7 @@ @SuppressWarnings("unchecked") public String deactivate() { - Command cmd = CommandsManager.getInstance().getCommand(CommandsManager.DEFAULT_CATALOG, "deactivate"); + Command cmd = CommandsManager.getInstance().getCommand("media", "deactivate"); Context context = MgnlContext.getInstance(); context.put(Context.ATTRIBUTE_REPOSITORY, MediaModule.REPO); This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |
From: <fg...@us...> - 2011-11-19 21:00:17
|
Revision: 3696 http://openutils.svn.sourceforge.net/openutils/?rev=3696&view=rev Author: fgiust Date: 2011-11-19 21:00:10 +0000 (Sat, 19 Nov 2011) Log Message: ----------- MEDIA-271 MediaFolderViewPage: media search always performed twice Modified Paths: -------------- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java Modified: trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java =================================================================== --- trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2011-11-19 17:14:59 UTC (rev 3695) +++ trunk/openutils-mgnlmedia/src/main/java/net/sourceforge/openutils/mgnlmedia/media/pages/MediaFolderViewPage.java 2011-11-19 21:00:10 UTC (rev 3696) @@ -243,9 +243,16 @@ if (result.getTotalSize() > 0 && result.getNumberOfPages() < page) { page = 1; + result = module.getSearch().search( + request, + type, + path, + false, + SortMode.valueOf(sorting), + itemsPerPage, + page); } - result = module.getSearch().search(request, type, path, false, SortMode.valueOf(sorting), itemsPerPage, page); return result; } This was sent by the SourceForge.net collaborative development platform, the world's largest Open Source development site. |