Closed (fixed)
Project:
Simplenews
Version:
7.x-1.x-dev
Component:
Code
Priority:
Normal
Category:
Bug report
Assigned:
Unassigned
Reporter:
Created:
6 Apr 2010 at 12:51 UTC
Updated:
4 Nov 2010 at 23:50 UTC
Simplenews can be installed and enabled even if tyxonomy is disabled, which results in some strange errors.
Comments
Comment #1
sutharsan commentedNot dependent on taxonomy is by design. Please specify the errors.
Comment #2
jurgenhaasAren't the categories of the newsletter system being implemented on top of taxonomy? I thought so because when you see the block of a newsletter you get "Previous Issues" linking to http://www.example.com/taxonomy/term/0 which indicates this is linking to a taxonomy page, isn't it?
Comment #3
sutharsan commentedNo, that link is still there but not effective. I started by removing the taxonomy dependency out of simplenews. But I need to write a lot of code to do so, therefore I have second thoughts about it and might revert this change. Thanks anyway for reporting the error, but you will find a lot more. If you have time you can help with the D7 port. The todo list is in the release notes.
Comment #4
tyr commentedThere seems to be still some dependency on taxonomy module. At least during the installation routine, I'm getting the following error:
FieldException: Attempt to create field name <em class="placeholder">field_simplenews_term</em> which already exists, although it is inactive. in field_create_field() (line 277 of [...]/modules/field/field.crud.inc).(Here I deactivated taxonomy before installing simplenews. The error message may be a little bit different when taxonomy has never been activated on a particular drupal installation.)
When taxonomy module is active, the installation routine runs smoothly.
Comment #5
miro_dietikerSutharsan committed finally a change to revert the taxonomy independence.
This means that currently simplenews is dependent on taxonomy as newsletter base entity. So the code needs to be updated and the dependency added.
Committed dependency to dev.
Comment #6
miro_dietikerso fixed... :-)