mojomojo / mojomojo

A Catalyst & DBIx::Class powered Wiki.

MojoMojo -- A Catalyst & DBIx::Class powered Wiki

Website located at http://mojomojo.org/

Installation: See lib/MojoMojo/Installation.pod

After Installation: Run script/mojomojo_server.pl to test the application.

License: This is free software. You can redistribute it and/or modify it under the same terms as perl itself.

mojomojo's Issues

It should not be possible to add the same tag twice to a page

One can add the same tag repeatedly for a page (and probably for a photo as well).

IRC formatter looks crap

http://dev.catalyst.perl.org/wiki/humor - the colored exchange looks much better but was created manually. The IRC formatter looks bad because it aligns the text so it's not visually as easy to tell which nick said what; and because it doesn't auto-color nicknames.

Decouple Text::SmartyPants from the Textile formatter?

Looks like Text::SmartyPants could be used as well with Markdown.

Comments, and a person's profile are always formatted using Textile

...even if the default formatter is not Textile. See MojoMojo::Schema::Result::Person->*_formatted. Interests, movies and music are forcibly Textile-formatted.

What's up with the Linked from in the bottom nav?

What's it supposed to do? What if 10 pages link to the current one?

Running mojomojo_spawn_db when the database already exists generates an error

http://pastie.org/454973

Code with trailing spaces takes forever to highlight

http://pastie.org/454381

Empty span added at end of comment line while in lang="Perl"

The Syntax Highlighter plugin adds an empty span of class "kateComment" to then end of Perl comment lines

mojomojo 0.999031 badly interprets URL.

mojomojo 0.999031 interprets "http://someurl/Development%20Documentation" in HTTP URL as "http://someurl/Developmentȍocumentation"(http://someurl/Development%C8%8Documentation ) in the generated link.

[test link](http://someurl/Development%20Documentation)

insertTags for Edit Toolbar needs ported to new snytax

The insertTags of the Edit toolbar will need to change from the old snytax:

<pre lang="Perl">
use strict;
</pre>

to the new syntax:

{{code lang="Perl"}}
use Me;
{{end}}

format_content methods should use ACCEPT_CONTEXT, not have $c passed to them

See http://github.com/marcusramberg/mojomojo/blob/23f062d76eaf0378a20e854bcfe12a730f176140/lib/MojoMojo/Schema/ResultSet/Content.pm#L23

Diffs are hard to read

Example: http://dev.catalyst.perl.org/wiki/deployment.view?rev=22, click "See changes"

The Markdown in <div>s issue

Summary:

Markdown is designed to NOT interpret text within HTML block elements (like <div>, <table> etc.) as markdown. The rationale is that you may want to paste a whole blob of HTML from somewhere else (counter code, widgets etc.).

On the other hand, when adding any sort of grid-based structure to your page, you want the text inside <div>s to be interpreted as Markdown:

<div class="navbar">
* [Home](/)
* [Products](/products)
* [About](/about)
</div>

You also generally want Markdown to be parsed in <div>s for centering images or any other object in a <div>.

See the thread about this issue at http://n2.nabble.com/No-Markdown-in-%3Cdiv%3Es-or-%3Ctable%3Es---tt2682337.html .

Currently, MojoMojo uses an option markdown_in_html_blocks => 1 in MojoMojo::Formatter::Markdown. This is bound to cause confusion, as different wiki users, who do not see the MojoMojo::Formatter::Markdown code, will assume different things about Markdown being interpreted in <div>s or not. Especially when pasting outside HTML, it can be hard to catch the errors resulting from Markdown parsing the stuff in HTML block elements.

The proposal is to have <div markdown="1">, which is also implemented in PHP Markdown Extra.

blockquotes are broken

The following markup should create a blockquote:

> quoted stuff

This is new in 0.28. Added tests for it to formatter_all_markdown.t.

Angle brackets (<less than>, <greater than>) are not rendered in code sections

Test case:

<xml>
    <foo>

You should see two lines in the code section above. Older MojoMojo handles this correctly: http://wiki.dandascalescu.com/test#Angle_brackets_.2F_less_than

This happens with either Markdown or Textile, even if Defang.pm and Pod.pm are removed from Formatters.

Textile test:

bc. <b>

This will bolden the rest of the text instead of being rendered as <b>.

MojoMojo doesn't clean up temp files

Reported by mst (major annoyance for Catwiki), Khisanth (Cache::FastMmap 1.34), dandv (/tmp/mojomojo/session and /tmp/sharefile-* take about 500Mb each, after ~100 MojoMojo restarts and relatively low wiki usage).

Possible solutions at http://www.perlmonks.org/?node_id=379922

Cache::FastMmap seems to better take care of removing temp files on exit since v1.31 but Khisanth still reports the issue.

Wikilinks that involve the parent path don't work

[[../bar]] from foo/ will resolve to __/bar

Creation of two or more non-existent pages in a row is broken

Steps to reproduce:

1. Go to http://mojomojo.org/test/new1/new2
2. You will be greeted with a page that says "Click on any non-existent page to create it. Any missing pages above it will be created automatically."
3. Click "new2" and create the page with some content.
4. Go to http://mojomojo.org/test/new1/
5. Enter some content there and click "Create"/
6. You will get an error page with the URL http://mojomojo.org/test/new1.test/new1, saying that the URL was not found. Also, the URL displayed does not match the URL of the page.

Markdown, Syntax Highlight and Lines Starting with #

Doesn't play well with

!/usr/bin/perl

or lines in general starting with # like a comment:

Aquesta filla és mamalluda.

Remove superfluous secondary navbar

On each page, there is a top navbar displaying just "Page".

Add a real delete page feature

Since this is a wiki, it should not actually expunge the page entirely.

Instead, it should do the following:

• Mark the page as deleted in the database
• Remove the page from the search index
• Don't show the page in sub pages lists

What happens to existing links to the page? Probably nothing, just let them exist.

It should be possible to undelete a deleted page.

There could also be a separate expunge option (for administrators only) that actually deleted the page from the system entirely.

Switch from DBIx::Class::UTF8columns to DBIx::Class::ForceUTF8

DBIx::Class::ForceUTF8 applies UTF8Columns to all columns. Good future proofing.

Pretty much any crap prepending an attachment link will work

See http://wiki.dandascalescu.com/blog/php_vs_perl.edit for an example:

It does work, though.

Testing GFL

These lines are separated
by a linebreak in the output

But these
-- aren't

Syntax Highlighting Broke After Syntax Change Branch Merge

Shit like <pre lang="Perl"> and <pre lang="SQL"> used to bring color to otherwise mono-color text in <pre> spans. Now they don’t.

line breaks are scrubbed

The following markdown will NOT generate a line break:

Roses are red
<br/>Violets are blue

This is new as of 0.28. The scrubber seems to be eating up the <br>.

.search within a subtree never returns results

Test: http://mojomojo.org/documentation.search?search_type=subtree&q=test
vs. http://mojomojo.org/documentation.search?search_type=all&q=test

TOC creates the same link for a different section

If the same name is used for two different headings, the TOC plugin links both headings to the same anchor.

Kinosearch re-indexes incompletely

The following process seems to get a complete search index (for only a period of time).

remove files from index/
restart MojoMojo
request a page – then the index is rebuilt.

This process creates one _number.cfs file and a file named segments. The segments file (I believe) references the .cfs by its number.

However after a period of time (less than a day I’ve witnessed) another .cfs file is created and segments is updated to reference it. The problem is that the newly created .cfs file is not complete. I can see this by the reduction in bytes compared to the original. In addition searches on certain terms don’t get as many hits as before. In other words, the newer index does not included all pages.

Edit summaries

Edit summaries are required for any serious wiki.

TWiki started without and finally smartened up in the very recent versions and implemented them. They are very important on Wikipedia, and a feature to be checked on the wiki comparison engine Wikimatrix.

Looks like we can map the edit summary onto the content.abstract field and just need the UI for it.

Export takes 100% CPU

http://dev.catalystframework.org/wiki/.export_html may crash Perl. On both Ubuntu and Windows, exporting takes 100% CPU.

KinoSearch is horrendously slow: 17 seconds for 15 results

Test: http://mojomojo.org/documentation.search?search_type=all&q=test

Repeatedly takes about 17 seconds to find 50 results. This is hardly acceptable.

.search ignores the site preference for Google search

Test: http://localhost:3000/reviews/software.search?q=dvd&search_type=all
vs. searching for "DVD" from the navbar, which shows the site preference for Google search.

A justification would be that Google couldn't do subtree searches, but still. With KinoSearch being so slow (see issue #32), letting the internal search run would be a DOS.

insert italics with markdown reloads page instead

When I use the toolbar to insert italics with markdown, mojomojo reloads the page instead. On the contrary, this doesn't happen with inserting of bold.

Syntax highlight doesn't recognize end of Perl POD

Once Perl POD is found, the syntax highlighter stays in that mode (kateComment class) even when the POD has ended and code started again.

There needs to be a way to disable unwanted plugins

MojoMojo::Formatter::Emote, for example transforms smileys into emoticons, and is called automatically by Module::Pluggable::Ordered.

Incorrect URL echoed in the error page

Test: http://mojomojo.org/advocacy.documentation

Result: "The requested URL was not found: http://mojomojo.org/documentation"

Split edit not available if user is not logged in

Split edit should be available for anonymous users as well.

Tag cloud is incorrectly calculated

All tags are at maximum size, and they don't care about number of pages per tag.

See http://wiki.dandascalescu.com/.tags and compare 'Perl' vs 'Romania'

Subpages of a page include itself, and display the page path instead of its title

Test: http://wiki.dandascalescu.com/howtos/catalyst/using_dbicx::testdatabase_with_catalyst.list

The output is

All pages in using_dbicx::testdatabase_with_catalyst listed alphabetically

    * Using_DBICx::TestDatabase_with_Catalyst

Permissions are not checked properly for rollback

If there is an anonymous username set in the database, anyone can roll a page back, even if anon users don't have edit perms!

The PageAdmin->auto method is just broken, and needs to check perms like ->edit does.

Review MojoMojo::Formatter::File

The whole File formatter is a bit dubious:

• the documentation appears to be incorrect (only the "filename" parameter is mentioned for =file, but the code parses a second plugin parameter)
• what is the base path to the file?
• the code doesn't do any checks against attempts to render /etc/passwrd
• there's a leftover Test.pm
• what is the use case for this exactly? That should go in the documentation

Email notification of page updates

Easy:

• notify the administrator(s) whenever a page is modified
• notify all editors of a page whenever a page was modified

Not so easy:

• do the above if a setting is present - may require schema changes

More difficult

• allow anyone (with permissions) to subscribe to a page changing. TWiki does this via... err... some settings page somewhere. I used to know this but I've been searching for it for the past 5 minutes. It's neither in the user's page, nor in WebPreferences, nor in the web's home page. Hint: don't do it that way.

Unify user Preferences and Profile page

Currently, Profile duplicates "Name" and "E-mail" from Preferences

Double scrollbar in the user search autocomplete drop-down

1. Have a site with many users (~30 here)
2. Go to roles
3. Type 'e' or another common character that's bound to match many usernames
4. If there are sufficient results, the scrollbar to the right of the dropdown will be doubled.

This may happen to any autocomplete, including page and photo tags.

The "Authors" link doesn't care about the page

Test: http://mojomojo.org/features.users will output all authors for the main page, not just those who edited "Features".

I've got issues

I’m gay but I don’t even like boys.

Backlinks are not calculated correctly for new pages

Let's say we have a link from /TopicA to /TopicB, and the /TopicB page does not exist. When we create the /TopicB page, the link table will not be populated until any page linking to /TopicB is edited and saved. This means backlinks are often wrong.

One fix is to store backlinks as from a page id to a page name.

Actions for nonexistent pages silently fall back to applying the action for /

Tests:
http://mojomojo.org/theresnopagelikethis.list
http://mojomojo.org/theresnopagelikethis/documentation.subtree etc.

Pages with 1 revision show 3 revisions in .info

Example: http://dev.catalyst.perl.org/wiki/thetutorial.info