Drupal 7 module
Project page: http://drupal.org/sandbox/rasmus/1928276

The Subviews module delivers an output style for Views that lets you create table like trees where views can be nested into eachother.

The output style contains the following features:
- expand/collapse functionality
- optional AJAX loading
- parsing field values as arguments to the subview
- remembering the expand/collapse state of a row in the user session
- mixed entity views. The nested views doesn't have to be the same entity type as the parent view.

Example ussages
- Taxonomy trees
- Taxonomy lists that display related content
- Node views with expandable comment lists
- Use it with Entity Reference to show related content

You need Views 3 for this module to work.

Git clone:
git clone http://git.drupal.org/sandbox/rasmus/1928276.git subviews

Reviews I have done so far:
http://drupal.org/node/1943788#comment-7185592
I know I need to do more and I will do them within the next couple of weeks

Comments

mayank-kamothi’s picture

mayank-kamothi commented
Status: Needs review » Needs work

hi

Autometed Review

Check this url http://ventral.org/pareview/httpgitdrupalorgsandboxrasmus1928276git

Thanks,
Mayank Kamothi

arun ak’s picture

arun ak commented

Hi mayank-kamothi,

I tried your module, I create a view with format Subview, then I add Recent comment block as subview.
Then in Subview General settings 'AJAX enabled expand/close' as Subviews behavior.

But I felt an issue, the 'AJAX enabled expand/close' functionality only working for first item on the list and the functionality not attach with remaining items in list.

Rasmus Steen’s picture

Rasmus Steen commented

ARUN AK: That is because you need to set up a contextual filter on your comments view and then have a Node ID field on your parent Subviews view. The Node ID field should be passed in as an argument to the comment view.

The reason why it is only attached to the first view is because Subviews identifies the views using a combination of the passed arguments. In this way it is possible for Subviews to remember the collapse/expand state across multiple Subviews for easy navigation.

Rasmus Steen’s picture

Rasmus Steen commented

ARUN AK: I will make a validation to check that you have set up the arguments if you are using the Expand/Collapse modes

Rasmus Steen’s picture

Rasmus Steen commented

mayank-kamothi: Code is now fixed according to pareview almost everywhere
ARUN AK: Expand/collapse functionality now only works when you have set up arguments

Rasmus Steen’s picture

Rasmus Steen commented
Status: Needs work » Needs review

Updated status

PA robot’s picture

PA robot commented

We are currently quite busy with all the project applications and I can only review projects with a review bonus. Please help me reviewing and put yourself on the PAReview: review bonus high priority list. Then I'll take a look at your project right away :-)

PA robot’s picture

PA robot commented
Issue summary: View changes

Changed the GIT clone command

mayank-kamothi’s picture

mayank-kamothi commented

Hi ARUN AK,

First see module developer name and than put comment like "I tried your module".
This is not my module and i have just given a review of this module.
I hope next time you will be a care full.

Thanks,
Mayank kamothi.

sreynen’s picture

sreynen commented
Title: Subviews » [D7] Subviews
kscheirer’s picture

kscheirer
Primary language English
Location Vallejo
commented
Status: Needs review » Needs work

Your project page could really use an image to show the module in action - just reading the text was not clear to me.

All your menu items in subviews_menu() have an access callback of TRUE, meaning they will work for any user and combined with _subviews_load_view() using whatever is present in a GET string could expose all views to the public.

Views already has a permission-based access system, ajax, and methods for dealing with user arguments. Please use them and close these security holes.

----
Top Shelf Modules - Enterprise modules from the community for the community.

PA robot’s picture

PA robot commented
Status: Needs work » Closed (won't fix)

Closing due to lack of activity. Feel free to reopen if you are still working on this application.

I'm a robot and this is an automated message from Project Applications Scraper.

PA robot’s picture

PA robot commented
Issue summary: View changes

Updated the list of reviews