Simplenews can be installed and enabled even if tyxonomy is disabled, which results in some strange errors.

Comments

sutharsan’s picture

Not dependent on taxonomy is by design. Please specify the errors.

jurgenhaas’s picture

Aren'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?

sutharsan’s picture

No, 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.

tyr’s picture

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

miro_dietiker’s picture

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

miro_dietiker’s picture

Status: Active » Fixed

so fixed... :-)

Status: Fixed » Closed (fixed)

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