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) {
#4 remove_support_for_editablefields_module-1780880-0.patch3.04 KBChi
PASSED: [[SimpleTest]]: [MySQL] 2,455 pass(es).
[ View ]


longwave’s picture

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

Title:Support for editablefields module» Fix or remove support for editablefields module
Category:support» task
longwave’s picture

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

Chi’s picture

Title:Fix or remove support for editablefields module» Remove support for editablefields module
Status:Active» Needs review
Issue tags:+needs backport to D6
new3.04 KB
PASSED: [[SimpleTest]]: [MySQL] 2,455 pass(es).
[ View ]

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

Status:Needs review» Fixed

Committed to both branches, thanks.

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