lml / quadbase

Quadbase.org - Question Banking Made Easy
quadbase.org
Other
18 stars 11 forks source link

Replaced link_to_function and disable_with #168

Closed zapcannon99 closed 12 years ago

zapcannon99 commented 12 years ago

This should get rid of all the deprecation errors fixes issue #120

Dantemss commented 12 years ago

Make sure you go through and test all those and that everything works, then it should be ready to be pulled.

Dantemss commented 12 years ago

Here you go Issue #171

zapcannon99 commented 12 years ago

mkay so other than that problem, the data is saved in the database

jpslav commented 12 years ago

@Dantemss - status?

Dantemss commented 12 years ago

@jpslav The changes look good & all tests pass, although I don't know if the tests work for these. Still, the deprecation messages are gone.

zapcannon99 commented 12 years ago

When I click on the buttons, I checked if they were still in the database and they were. Just to make sure though, I will do a second check to make sure if the buttons do their job correctly. However, I will not be able to test the dependencies, since they don't function properly in the first place, but that is another issue to be fixed. Hopefully it won't cause any merge conflicts.

zapcannon99 commented 12 years ago

yeah buttons interact with the database this is good to go

Dantemss commented 12 years ago

@jpslav all good