Backport of the crud file from D6 Version
gordon - May 4, 2008 - 12:39
| Project: | Content Construction Kit (CCK) |
| Version: | 5.x-1.x-dev |
| Component: | content.module |
| Category: | feature request |
| Priority: | normal |
| Assigned: | gordon |
| Status: | needs review |
Description
To help with testing http://drupal.org/project/issues/48429 I have back ported the crud file from the drupal 6 version.
| Attachment | Size |
|---|---|
| cck-crudbackport.patch | 33.81 KB |
#1
We have added these patches to Lonely Planets system, and we found a bug with deleteing fields.
Here is a new patch
#2
I have made an update to this patch, so that all the field updates use the crud file like they do in D6. I also found and fixed the problem with the creation of the tables a second time.
#3
Could you please explain what bugs you found and fixed?
#4
The bug that I found was actually in the crud file in that when a new instance of a field was found it would try and create the table a second time.
#5
Ok, I found some more problems with the system trying to alter tables twice. I think this is complete now. However there is a couple of failed tests which I am going cross eyed looking at.
#6
I have found the last bug in the crud file and it is now passing *all* the tests that have been backported from Drupal 6.
#7
Oops, I added additional changes that were already committed.
#8
I found a bug with one of the tables which caused some fields not to save data.
#9
Gordon, many thanks for your work on this. We're not forgetting it, but we're currently focusing on a D6 RC for now.
Just a quick note to make sure you saw that D6 CRUD API commit : http://drupal.org/cvs?commit=123933
#10
No Problem, Just for me at LP, we just can't use D6 yet maybe after the new year we will be able to upgrade, but until then we need CCK to be at it's best.
And these patches I believe that it will make CCK last until D5's end of life.
#11
Does the 2008-07-25 5.x freeze mean this feature will not make it in?
#12
Here is an update for the new version of CCK