The default views that come with this module use the 'language' field.

However, this field is only conditionally defined in hook_views_data(), dependent on locale module being enabled.

This means that if local module is not present, Views is trying to show a field whose handler cannot be found, and therefore outputs error messages.

Original report

It occours using Legal 7.x-1.2 and Views 7.x-3.x-dev (as of 26-oct-2011)

Comments

marcoka’s picture

marcoka commented
Status: Active » Postponed (maintainer needs more info)

yea ok...well please provide the error output. thank you.

mandreato’s picture

mandreato commented

Field "Legal: Additional Checkboxes" on views has a broken handler.
It occours using Legal 7.x-1.2 and Views 7.x-3.x-dev (as of 26-oct-2011).

marcoka’s picture

marcoka commented

uhm. you wrote that already. i asked you how to reproduce the error

mandreato’s picture

mandreato commented

- install a fresh D7 environment
- install ctools 7.x-1.0 + views 7.x-3.3+108-dev (as of today) + legal 7.x-1.x-dev (as of today)
- on admin/config/people/legal set display style to "page link", then add "test" on Terms & Conditions, then add a "test label" under additional checkboxes
- on admin/structure/views, add a new view which show "Legal terms and conditions"
- try to add a "Legal: Additional Checkboxes" field, it will have "Broken/missing handler"

muschpusch’s picture

muschpusch commented
Status: Postponed (maintainer needs more info) » Active

Just run: drush views-dev

Views will create this output on any view (views backend):

Missing handler: legal_conditions language field
_views_create_handler - loading handler handler(handler type: field) failed: class legal_views_handler_field_unserialized_list could not be loaded. Verify the class file has been registered in the corresponding .info-file (files[]).
Missing handler: legal_conditions extras field
_views_create_handler - loading handler handler(handler type: field) failed: class legal_views_handler_field_exploded_list could not be loaded. Verify the class file has been registered in the corresponding .info-file (files[]).
Missing handler: legal_conditions changes field
Missing handler: legal_conditions language sort
Missing handler: legal_accepted language field
Missing handler: legal_accepted language sort

sdrycroft’s picture

sdrycroft commented
Version: 7.x-1.2 » 7.x-1.4

By default the legal_users and legal_terms views defined in legal.views_default.inc both include a sort on language which will result in a Missing handler error if the locale module is not installed. There are other issues in this file, as visiting "admin/structure/views/view/legal_users" and "admin/structure/views/view/legal_terms" even with Locale installed results in Broken/missing handlers on the page (and the "All changes are stored temporarily. Click Save to make your changes permanent. Click Cancel to discard your changes." message which is only displayed when views has had to clean up a messy view).

sdrycroft’s picture

sdrycroft commented

Has there been any progress with this?

joachim’s picture

joachim commented
Title: Field "Legal: Additional Checkboxes" on views has a broken handler » Default views make use of language field which is missing if Locale module is not present
Issue summary: View changes

Updated title and summary.

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Version: 7.x-1.4 » 7.x-1.x-dev

This sounds like a simple dependency line in the info file should solve this.

sdrycroft’s picture

sdrycroft commented

I don't think adding the Locale module as a dependency is a good fix for this. The default views should instead check for the presence of the locale module and change the view accordingly. No functionality of the module requires the Locale module, so adding it as a dependency just to fix a badly structured view is not a good fix.

robert castelo’s picture

robert castelo commented
Status: Active » Fixed

I think this was fixed a while back as part of another issue by adding a few if (module_exists('locale')) { conditions to the legal.views.inc file.

If anyone still experiencing this please re-open.

Status: Fixed » Closed (fixed)

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