Modify

Opened 20 months ago

Closed 16 months ago

Last modified 15 months ago

#10706 closed defect (fixed)

New install will not be possible with Trac 1.0

Reported by: rjollos Owned by: hasienda
Priority: normal Component: VotePlugin
Severity: normal Keywords: install db rollback
Cc: hasienda Trac Release: 1.0

Description

This plugin most likely won't work in Trac 1.0 for the reasons described in #9521. The plugin needs to query the table to get the schema version from the system table rather than relying on a check for the existence of the votes table, which . The latter results in an exception when the table is not present, which is an approach that no longer works with Trac 1.0.

Attachments (0)

Change History (9)

comment:1 follow-up: Changed 20 months ago by hasienda

  • Keywords install db rollback added

Hey, but the experience gained by working on #9521 should make the fix nearly a no-brainer, until I get around to do the rework I dreamed about earlier this year.

Do you plan to fix it yourself?

comment:2 in reply to: ↑ 1 ; follow-up: Changed 20 months ago by rjollos

  • Status changed from new to assigned

Replying to hasienda:

Do you plan to fix it yourself?

Yeah, I'm going to use it as an opportunity to study and better understand your fix. I'll certainly appreciate if you have a chance to code review my patch though!

comment:3 in reply to: ↑ 2 Changed 19 months ago by hasienda

Replying to rjollos:

...
I'll certainly appreciate if you have a chance to code review my patch though!

Sure.

comment:4 follow-up: Changed 16 months ago by hasienda

Just testing this plugin for support of user ID changes in AccountManagerPlugin.

Thanks to modifications of Trac's environment upgrade detection code (giving every setup participant an own db connection instead of passing one around) this plugin works with Trac 1.0 too.

Still we should introduce schema tracking in system db table. And there are more improvements, that would justify moving to a new db schema. But I'll open a new ticket for that enhancement request for the sake of better tracking.

comment:5 in reply to: ↑ 4 Changed 16 months ago by rjollos

Replying to hasienda:

Thanks to modifications of Trac's environment upgrade detection code (giving every setup participant an own db connection instead of passing one around) this plugin works with Trac 1.0 too.

Oh, good to know. I didn't realize that a fix had been put in place prior to the 1.0 release.

comment:6 Changed 16 months ago by hasienda

  • Owner changed from rjollos to hasienda
  • Status changed from assigned to new

comment:7 Changed 16 months ago by hasienda

  • Resolution set to fixed
  • Status changed from new to closed

(In [12773]) TracVote: Don't call rollback() from environment_needs_upgrade(), closes #10706.

This is patterned after [12124] for TagsPlugin. See #9521 for details on
implications of probing the Trac db for table existence like done before.

comment:8 Changed 16 months ago by rjollos

I see that you tagged 0.1.4. Great! Thanks for fixing these issues.

comment:9 Changed 15 months ago by hasienda

(In [13090]) VotePlugin: Finish version nummer corrections after [13088:13089], refs #10706.

Add Comment

Modify Ticket

Action
as closed .
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from hasienda. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.