Posted by pthomsen on January 30, 2007 at 10:20pm
| Project: | Textimage |
| Version: | 5.x-2.x-dev |
| Component: | Miscellaneous |
| Category: | bug report |
| Priority: | critical |
| Assigned: | Unassigned |
| Status: | closed (fixed) |
Issue Summary
I'm using pgSQL as my database and have upgraded to 5.1 today. During the upgrade I got an error in the SQL script that performs the database changes. I don't have the exact error message, but the problem was related to the 'CREATE TABLE' statement. A field named 'presetid' was used in a couple of places. That field does not exist. I changed the code (see patch below), which resolved the problem.
| Attachment | Size |
|---|---|
| textimage-5.patch | 590 bytes |
Comments
#1
Oops. Since I have a patch, I changed the status.
#2
improved patch that takes care of uninstall and update too
#3
Patch updated to create and drop the correct sequence name. Tested with Drupal 5.1 and PostgreSQL 8.1, works like a charm after making the fonts directory writable for Apache (see http://drupal.org/node/111838)
#4
Yes, I have the same problems.
One thing, I propose to change pid column type to int_unsigned and remove superfluous NOT NULL (since the column is PK already).
#5
Here is a further patch, which fixes both problems. I can't see a reason not to commit this, since the bug is stopping people from using the module without complicated manual mucking around.
This patch differs from the previous ones in that it:
- moves the "PRIMARY KEY" into the field spec and removes the redundant "NOT NULL"
- replaces the checks of the fonts_dir & images_dir with is_dir(), since these directories do not need to be writable. (i.e. also fixes http://drupal.org/node/111838 )
Other than this it is the same as the previous patch.
I have tested this, and it seems to work.
#6
Patch is out of date (no fault of patch contributors) and needs a re-write.
Patch should only contain fixes related to this issue, any other fixes should be posted to a separate issue.
As I don't know Postgres I do hope that someone will take the time to submit a working patch, and I will be certain to make sure it goes into the code.
#7
Fixed in DRUPAL-5--2.
#8
Automatically closed -- issue fixed for 2 weeks with no activity.