radiant / radiant-event-calendar-extension

Create, organise and display calendar events in your radiant site. Read, aggregate and serve ical subscriptions.
37 stars 13 forks source link

Typo? #18

Closed rosslaird closed 13 years ago

rosslaird commented 13 years ago

I was having an issue with this extension:

ActionView::TemplateError (Mysql::Error: Table 'rl_radiant.event_venues' doesn't exist: SELECT * FROM event_venues ORDER BY title asc) on line #6 of vendor/extensions/event_calendar/app/views/admin/events/_form.html.haml:

I tried to uninstall it -- but when I did, I got an error with this spelling:

undefined method `add_colum' for #<ActiveRecord:...

Shouldn't that be "add_column", with an "n"? Maybe this is the part of the source of my problem with the extension?

KevinTriplett commented 13 years ago

You're right, there's a typo, but I don't think that was the original problem. It would only affect the down migration when uninstalling. Here's a patch for it:

https://github.com/KevinTriplett/radiant-event-calendar-extension/commit/2157065698f701b092589d6e21971710654f3b68

or you can fix it manually by editing the db/migrate/008_clean_out_calendar.rb file.

It sounds like the original problem is complaining that the event_venues table does not exist. You might check to make sure that the table was created in your database when you ran the migration -- file db/migrate/20100219102227_venues_and_categories.rb creates that table.

rosslaird commented 13 years ago

Thanks for the feedback. Yup, that did it. My main problem, I think, has to do with the sass stylesheet, which (I'm trying to recall the exact error message) has a token error. Ah, here it is:

Sass::SyntaxError (Expected rparen token, was ident token.): public/stylesheets/sass/event_calendar.sass:43 public/stylesheets/sass/event_calendar.sass:43

KevinTriplett commented 13 years ago

Ah okay -- looking in the sass file around line 43 I don't see the error although the comment on line 46 looks suspicious in that it doesn't end with a forward slash:

  /*** calendars and maps *******
rosslaird commented 13 years ago

Yes, I imagine that would do it. Probably the error backs up to the beginning of the rule. I don't know sass, but in css an open comment typically causes the browser to choke on the whole rule block. (And an open block, without a closing brace, can cause the remainder of the whole css file to fail.)

KevinTriplett commented 13 years ago

Sorry, I should have thought about this before responding -- the whole philosophy behind sass and haml is that you don't have to close anything, so that's probably not it.