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
Members fund testing for the Drupal project. Drupal Association Learn more


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
3.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.