Just wondering how it works. I didn't find this hook in views module.

/**
 * Implements hook_views_tables_alter().
 *
 * Conditionally adds editablefields support.
 */
function uc_product_views_tables_alter(&$tables) {
CommentFileSizeAuthor
#4 remove_support_for_editablefields_module-1780880-0.patch3.04 KBchi

Comments

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

This is legacy code from 5.x-1.x. I have not tested it in 7.x-3.x and am not convinced it will even work in 6.x-2.x.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Title: Support for editablefields module » Fix or remove support for editablefields module
Category: support » task
longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

See also #611038: Support for editablefields which suggests this code should just be removed.

chi’s picture

chi commented
Title: Fix or remove support for editablefields module » Remove support for editablefields module
Status: Active » Needs review
Issue tags: +Needs backport to D6
StatusFileSize
new remove_support_for_editablefields_module-1780880-0.patch3.04 KB

Here is the issue in views module #105620: Allow modules to alter the tables and arguments arrays. It was added to views_cache.inc but the file has been removed later.
Here the commit: http://drupalcode.org/project/views.git/commitdiff/582713d953d8c04ee4d42...
So lets remove this hook. I think it should be fixed in editablefields module with hook_views_data_alter().

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Needs review » Fixed

Committed to both branches, thanks.

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