to work "profile" with "views" I had to change the name of the table profile_value to profile_values
| Comment | File | Size | Author |
|---|---|---|---|
| #8 | profile.views_convert.inc_.patch | 593 bytes | jmaties |
| #7 | profile.views_convert.inc_.patch | 593 bytes | jmaties |
| #2 | profile.patch | 1.59 KB | jmaties |
Comments
Comment #1
damien tournoud commentedThis is called a bug in Views, then :)
Comment #2
jmaties commentedSorry, it is in views :P
--- views-7.x-3.x-dev.tar.gz/views/modules/profile.views.inc
+++views-7.x-3.x-dev.tar.gz/views/modules/profile.views.inc
@@ -17,16 +17,16 @@
function profile_views_data() {
// Define the base group of this table. Fields that don't
// have a group defined will go into this field by default.
- $data['profile_values']['table']['group'] = t('Profile');
+ $data['profile_value']['table']['group'] = t('Profile');
- $data['profile_values']['table']['join'] = array(
+ $data['profile_value']['table']['join'] = array(
'node' => array(
- 'left_table' => 'profile_values',
+ 'left_table' => 'profile_value',
'left_field' => 'uid',
'field' => 'uid',
),
'users' => array(
- 'left_table' => 'profile_values',
+ 'left_table' => 'profile_value',
'left_field' => 'uid',
'field' => 'uid',
),
@@ -34,20 +34,20 @@
$fields = profile_views_get_fields();
foreach($fields as $field) {
- $table_name = 'profile_values_' . $field->name;
+ $table_name = 'profile_value_' . $field->name;
$data[$table_name] = array(
'table' => array(
'group' => t('Profile'),
'join' => array(
'node' => array(
- 'table' => 'profile_values',
+ 'table' => 'profile_value',
'left_table' => 'users',
'left_field' => 'uid',
'field' => 'uid',
'extra' => array(array('field' => 'fid', 'value' => $field->fid)),
),
'users' => array(
- 'table' => 'profile_values',
+ 'table' => 'profile_value',
'left_field' => 'uid',
'field' => 'uid',
'extra' => array(array('field' => 'fid', 'value' => $field->fid)),
Comment #3
esmerel commentedComment #4
dawehnerThis is fixed.
Next time post a patch with a usable header :)
Comment #5
jmaties commentedsorry is my first patch ;)
Comment #6
dawehnerNo problem :)
Thanks for finding and fixing the bug
Comment #7
jmaties commentednew modification and new patch ;)
Comment #8
jmaties commentedsorry the correct patch
Comment #9
dawehnerI will not commit this. I don't think views will support d5 -> d7 without converting it to d6 first.
There will be a alternative api, see issue queue, to convert from d6 views to d7 views, but this patch here will not fix it, sry.
I hope it's okay for you.
Comment #10
dawehnerHere is the other issue: #687564: Build a generic d6 -> d7 update api / converter
Comment #11
jmaties commentedok, thank's