Bug #2256
Various problem using trunk on Redmine 2.3.3
100%
Description
I tried the trunk version on a new Redmine 2.3.3 installation and ran into the following problems:
- the
WikiMacro.register_all!
call at the end ofinit.rb
causes an exception when trying to runrake redmine:plugins:migrate
when the new wiki_macros migration has not been run before (and therefor the table for the model does not exist yet) - the missing support for the (new?)
text
argument for wiki macros caused the macro replacement code to crash - the
exec_macro_with_custom
hook inwiking_macros_definitions_patch.rb
does not get triggered when a WikiNG custom macro has not been loaded yet (since I removedWikiMacro.register_all!
to prevent the problems with the migration) because the wiki formatting code callsRedmine::WikiFormatting::Macros::Definitions#macro_exists?
before callingexec_macro
.
I created a patch that solved all my problems which you can find attached.
The patch does not take different Redmine version into account.
History
#1 Updated by Andriy Lesyuk about 11 years ago
Thank you very much, Martin! In fact, the SVN version is not intended to be functioning yet!.. It’s the version in progress. Anyway your findings will definitely help me! Thanks!
P.S. For a stable WikiNG use the file: wiking_0.0.4.tar.bz2.
#2 Updated by Andriy Lesyuk about 11 years ago
- Status changed from New to Open
- Assignee set to Andriy Lesyuk
- Target version set to 1.0.0b
#3 Updated by Martin Corino about 11 years ago
Andriy Lesyuk wrote:
Thank you very much, Martin! In fact, the SVN version is not intended to be functioning yet!.. It’s the version in progress. Anyway your findings will definitely help me! Thanks!
P.S. For a stable WikiNG use the file: wiking_0.0.4.tar.bz2.
Yes, I was aware of that, but I was very interested to see the custom macro functionality so I just decided to give it a try. For me it works good enough for now.
I have your repository cloned in Git so I can easily merge your changes as they come.
#4 Updated by Andriy Lesyuk about 11 years ago
- Due date set to 19 Oct 2013
- Status changed from Open to Closed
- % Done changed from 0 to 100
Your fixes helped, Martin. Thank you, again!