I am working on a 6.x version. It currently works but needs a bit more polish and support for Views 2 conversion.

CommentFileSizeAuthor
#10 editview-367744-10_HEAD.patch77.87 KBdarren oh
#9 editview.tar_.gz9.13 KBdarren oh
#6 editview-367744-6_HEAD.patch77.67 KBdarren oh
#5 editview-367744-5_D6.patch72.43 KBdarren oh
#4 editview-D6.patch66.48 KBdarren oh

Comments

Anonymous’s picture

Anonymous (not verified) commented

Sounds good!

What do you mean by "Views 2 conversion" though? Just conversion of Views 1.x code to Views 2.x code, or something more?

How soon do you reckon you'll be ready to at least commit some code? I'd like to use this module :-)

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented

Views 2 cannot use Views 1 views unless they are first converted. Module authors have to provide the code for the conversion.

antiorario’s picture

antiorario commented

subscribing

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Active » Needs work
StatusFileSize
new editview-D6.patch66.48 KB

Here's an initial patch. It does not yet support hook_field_form_render() or upgrading views from Views 1 to Views 2.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
StatusFileSize
new editview-367744-5_D6.patch72.43 KB

Just needs to support upgrading to Views 2 now.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
Status: Needs work » Reviewed & tested by the community
StatusFileSize
new editview-367744-6_HEAD.patch77.67 KB

Finished. Upgrading old views will work as soon as Views 2 is fixed. See http://drupal.org/node/316556#comment-1242627 for patch.

agileware’s picture

agileware commented
Status: Reviewed & tested by the community » Needs review

Thanks for the patch Darren - good job.

Assigning correct status, see http://drupal.org/node/156119. So lets get some community review and testing.

Anonymous’s picture

Anonymous (not verified) commented

I'm not really in a position to test this while it's a patch, could you please upload a tarball as well?

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
StatusFileSize
new editview.tar_.gz9.13 KB

Sure! Looking forward to a quick review.

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented
StatusFileSize
new editview-367744-10_HEAD.patch77.87 KB

Still no review? Attached patch adds uninstall support.

agileware’s picture

agileware commented

It's taking a while due to other things I've got going on sorry.

I have reviewed the patch and it is good and I will create a 6 version from it.

There are just a few things I am adding before I put it up:
*Support for attachments
*Support for date fields
*Support for image attach fields

I have almost done this so it shouldn't be too long. Hopefully within the next day.

Thanks for the patch.

chaloalvarezj’s picture

chaloalvarezj commented

Hi!
I have downloaded the drupal 6 version of this module and tried with a content type containing:
1 node reference field
1 content taxonomy field
2 integer fields
1 text field

I have set up a view to bring data from this content type by passing the node reference as argument. The display option was set to editview table but nothing happens.... Well I get three (number of actual data) empty rows wit update and cancel buttons below... but data is not displayed.

Am I doing something wrong?

darren oh’s picture

darren oh
Primary language English
Location Lakeland, Florida
commented

You may have forgotten to define some of the fields your view needs. The 6.x version should really only be tested by those who had the 5.x version working before upgrading to Drupal 6.

We're just waiting for Agileware to add support for some special fields, so no more testing is necessary.

agileware’s picture

agileware commented

When I tried the patch it didn't work for any cck fields. You would get the heading but no data on any rows. That is probably what chaloalvarezj is experiencing.

I have since fixed this as well as added support for image attach and file attachments.

I have one remaining bug with date cck fields that I have yet to be able to resolve but as soon as that is worked out I will put up at least a dev version.

chaloalvarezj’s picture

chaloalvarezj commented

Hi Agilware,
That is exactly my problem: all the fields where cck fields.
I am looking forward to the dev version!

Cheers

nterbogt’s picture

nterbogt commented

The 6.x-dev version is released! Go check it out and let us know of any problems you encounter.

There is one known issue with date fields but this seems to be an implementation of date widgets that is causing the problem. I'll bring this up with the date maintainer.

chaloalvarezj’s picture

chaloalvarezj commented

Hi! I have just downloaded the DEV version.. It's great!!! All my CCK fields are showing perfectly! The developer(s) did a great Job!!

I have however one problem: My content type is using the automatic node title module, so I cannot create a new node (row) without providing a node title value on editview.

How can i give a dummy title to the new node title field? so that no filling up is necessary.

Thank you!

agileware’s picture

agileware commented
Status: Needs review » Fixed

Hi chaloalvarezj,

The best way to track this issue would be to create a new issue against the 6.x-1.x-dev version with an appropriate title.

This will also make it easier for people to find if they have the same issue.

I am closing this issue as the port to 6.x version is done.

Thanks,

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.