Fossil SCM
Fix some (certainly not all) problems with wiki editing.
Commit
df646a7f4ca2562878294c860ef57c4a0a5b1afa
Parent
677aa71bca93b77…
2 files changed
+10
-10
+4
-3
+10
-10
| --- src/wiki.c | ||
| +++ src/wiki.c | ||
| @@ -133,19 +133,19 @@ | ||
| 133 | 133 | zBody = m.zWiki; |
| 134 | 134 | } |
| 135 | 135 | } |
| 136 | 136 | if( (rid && g.okWrWiki) || (!rid && g.okNewWiki) ){ |
| 137 | 137 | style_submenu_element("Edit", "Edit Wiki Page", |
| 138 | - mprintf("%s/wikiedit?name=%t", g.zTop, zPageName)); | |
| 138 | + mprintf("%s/wikiedit?name=%T", g.zTop, zPageName)); | |
| 139 | 139 | } |
| 140 | 140 | if( rid && g.okApndWiki ){ |
| 141 | 141 | style_submenu_element("Append", "Add A Comment", |
| 142 | - mprintf("%s/wikiappend?name=%t", g.zTop, zPageName)); | |
| 142 | + mprintf("%s/wikiappend?name=%T", g.zTop, zPageName)); | |
| 143 | 143 | } |
| 144 | 144 | if( g.okHistory ){ |
| 145 | 145 | style_submenu_element("History", "History", |
| 146 | - mprintf("%s/whistory?name=%t", g.zTop, zPageName)); | |
| 146 | + mprintf("%s/whistory?name=%T", g.zTop, zPageName)); | |
| 147 | 147 | } |
| 148 | 148 | zHtmlPageName = mprintf("%h", zPageName); |
| 149 | 149 | style_header(zHtmlPageName); |
| 150 | 150 | blob_init(&wiki, zBody, -1); |
| 151 | 151 | wiki_convert(&wiki, 0); |
| @@ -223,14 +223,14 @@ | ||
| 223 | 223 | db_multi_exec("INSERT OR IGNORE INTO unsent VALUES(%d)", nrid); |
| 224 | 224 | manifest_crosslink(nrid, &wiki); |
| 225 | 225 | blob_reset(&wiki); |
| 226 | 226 | content_deltify(rid, nrid, 0); |
| 227 | 227 | db_end_transaction(0); |
| 228 | - cgi_redirectf("wiki?name=%t", zPageName); | |
| 228 | + cgi_redirectf("wiki?name=%T", zPageName); | |
| 229 | 229 | } |
| 230 | 230 | if( P("cancel")!=0 ){ |
| 231 | - cgi_redirectf("wiki?name=%t", zPageName); | |
| 231 | + cgi_redirectf("wiki?name=%T", zPageName); | |
| 232 | 232 | return; |
| 233 | 233 | } |
| 234 | 234 | if( zBody==0 ){ |
| 235 | 235 | zBody = mprintf("<i>Empty Page</i>"); |
| 236 | 236 | } |
| @@ -248,11 +248,11 @@ | ||
| 248 | 248 | if( z[0]=='\n' ) n++; |
| 249 | 249 | } |
| 250 | 250 | if( n<20 ) n = 20; |
| 251 | 251 | if( n>200 ) n = 200; |
| 252 | 252 | @ <form method="POST" action="%s(g.zBaseURL)/wikiedit"> |
| 253 | - @ <input type="hidden" name="name" value="%t(zPageName)"> | |
| 253 | + @ <input type="hidden" name="name" value="%h(zPageName)"> | |
| 254 | 254 | @ <textarea name="w" class="wikiedit" cols="80" |
| 255 | 255 | @ rows="%d(n)" wrap="virtual">%h(zBody)</textarea> |
| 256 | 256 | @ <br> |
| 257 | 257 | @ <input type="submit" name="preview" value="Preview Your Changes"> |
| 258 | 258 | @ <input type="submit" name="submit" value="Apply These Changes"> |
| @@ -350,14 +350,14 @@ | ||
| 350 | 350 | db_multi_exec("INSERT OR IGNORE INTO unsent VALUES(%d)", nrid); |
| 351 | 351 | manifest_crosslink(nrid, &wiki); |
| 352 | 352 | blob_reset(&wiki); |
| 353 | 353 | content_deltify(rid, nrid, 0); |
| 354 | 354 | db_end_transaction(0); |
| 355 | - cgi_redirectf("wiki?name=%t", zPageName); | |
| 355 | + cgi_redirectf("wiki?name=%T", zPageName); | |
| 356 | 356 | } |
| 357 | 357 | if( P("cancel")!=0 ){ |
| 358 | - cgi_redirectf("wiki?name=%t", zPageName); | |
| 358 | + cgi_redirectf("wiki?name=%T", zPageName); | |
| 359 | 359 | return; |
| 360 | 360 | } |
| 361 | 361 | zHtmlPageName = mprintf("Append Comment To: %h", zPageName); |
| 362 | 362 | style_header(zHtmlPageName); |
| 363 | 363 | if( P("preview")!=0 ){ |
| @@ -369,11 +369,11 @@ | ||
| 369 | 369 | @ <hr> |
| 370 | 370 | blob_reset(&preview); |
| 371 | 371 | } |
| 372 | 372 | zUser = PD("u", g.zLogin); |
| 373 | 373 | @ <form method="POST" action="%s(g.zBaseURL)/wikiappend"> |
| 374 | - @ <input type="hidden" name="name" value="%t(zPageName)"> | |
| 374 | + @ <input type="hidden" name="name" value="%h(zPageName)"> | |
| 375 | 375 | @ Your Name: |
| 376 | 376 | @ <input type="text" name="u" size="20" value="%h(zUser)"><br> |
| 377 | 377 | @ Comment to append:<br> |
| 378 | 378 | @ <textarea name="r" class="wikiedit" cols="80" |
| 379 | 379 | @ rows="10" wrap="virtual">%h(PD("r",""))</textarea> |
| @@ -430,10 +430,10 @@ | ||
| 430 | 430 | "SELECT substr(tagname, 6, 1000) FROM tag WHERE tagname GLOB 'wiki-*'" |
| 431 | 431 | " ORDER BY lower(tagname)" |
| 432 | 432 | ); |
| 433 | 433 | while( db_step(&q)==SQLITE_ROW ){ |
| 434 | 434 | const char *zName = db_column_text(&q, 0); |
| 435 | - @ <li><a href="%s(g.zBaseURL)/wiki?page=%t(zName)">%h(zName)</a></li> | |
| 435 | + @ <li><a href="%s(g.zBaseURL)/wiki?page=%T(zName)">%h(zName)</a></li> | |
| 436 | 436 | } |
| 437 | 437 | db_finalize(&q); |
| 438 | 438 | style_footer(); |
| 439 | 439 | } |
| 440 | 440 |
| --- src/wiki.c | |
| +++ src/wiki.c | |
| @@ -133,19 +133,19 @@ | |
| 133 | zBody = m.zWiki; |
| 134 | } |
| 135 | } |
| 136 | if( (rid && g.okWrWiki) || (!rid && g.okNewWiki) ){ |
| 137 | style_submenu_element("Edit", "Edit Wiki Page", |
| 138 | mprintf("%s/wikiedit?name=%t", g.zTop, zPageName)); |
| 139 | } |
| 140 | if( rid && g.okApndWiki ){ |
| 141 | style_submenu_element("Append", "Add A Comment", |
| 142 | mprintf("%s/wikiappend?name=%t", g.zTop, zPageName)); |
| 143 | } |
| 144 | if( g.okHistory ){ |
| 145 | style_submenu_element("History", "History", |
| 146 | mprintf("%s/whistory?name=%t", g.zTop, zPageName)); |
| 147 | } |
| 148 | zHtmlPageName = mprintf("%h", zPageName); |
| 149 | style_header(zHtmlPageName); |
| 150 | blob_init(&wiki, zBody, -1); |
| 151 | wiki_convert(&wiki, 0); |
| @@ -223,14 +223,14 @@ | |
| 223 | db_multi_exec("INSERT OR IGNORE INTO unsent VALUES(%d)", nrid); |
| 224 | manifest_crosslink(nrid, &wiki); |
| 225 | blob_reset(&wiki); |
| 226 | content_deltify(rid, nrid, 0); |
| 227 | db_end_transaction(0); |
| 228 | cgi_redirectf("wiki?name=%t", zPageName); |
| 229 | } |
| 230 | if( P("cancel")!=0 ){ |
| 231 | cgi_redirectf("wiki?name=%t", zPageName); |
| 232 | return; |
| 233 | } |
| 234 | if( zBody==0 ){ |
| 235 | zBody = mprintf("<i>Empty Page</i>"); |
| 236 | } |
| @@ -248,11 +248,11 @@ | |
| 248 | if( z[0]=='\n' ) n++; |
| 249 | } |
| 250 | if( n<20 ) n = 20; |
| 251 | if( n>200 ) n = 200; |
| 252 | @ <form method="POST" action="%s(g.zBaseURL)/wikiedit"> |
| 253 | @ <input type="hidden" name="name" value="%t(zPageName)"> |
| 254 | @ <textarea name="w" class="wikiedit" cols="80" |
| 255 | @ rows="%d(n)" wrap="virtual">%h(zBody)</textarea> |
| 256 | @ <br> |
| 257 | @ <input type="submit" name="preview" value="Preview Your Changes"> |
| 258 | @ <input type="submit" name="submit" value="Apply These Changes"> |
| @@ -350,14 +350,14 @@ | |
| 350 | db_multi_exec("INSERT OR IGNORE INTO unsent VALUES(%d)", nrid); |
| 351 | manifest_crosslink(nrid, &wiki); |
| 352 | blob_reset(&wiki); |
| 353 | content_deltify(rid, nrid, 0); |
| 354 | db_end_transaction(0); |
| 355 | cgi_redirectf("wiki?name=%t", zPageName); |
| 356 | } |
| 357 | if( P("cancel")!=0 ){ |
| 358 | cgi_redirectf("wiki?name=%t", zPageName); |
| 359 | return; |
| 360 | } |
| 361 | zHtmlPageName = mprintf("Append Comment To: %h", zPageName); |
| 362 | style_header(zHtmlPageName); |
| 363 | if( P("preview")!=0 ){ |
| @@ -369,11 +369,11 @@ | |
| 369 | @ <hr> |
| 370 | blob_reset(&preview); |
| 371 | } |
| 372 | zUser = PD("u", g.zLogin); |
| 373 | @ <form method="POST" action="%s(g.zBaseURL)/wikiappend"> |
| 374 | @ <input type="hidden" name="name" value="%t(zPageName)"> |
| 375 | @ Your Name: |
| 376 | @ <input type="text" name="u" size="20" value="%h(zUser)"><br> |
| 377 | @ Comment to append:<br> |
| 378 | @ <textarea name="r" class="wikiedit" cols="80" |
| 379 | @ rows="10" wrap="virtual">%h(PD("r",""))</textarea> |
| @@ -430,10 +430,10 @@ | |
| 430 | "SELECT substr(tagname, 6, 1000) FROM tag WHERE tagname GLOB 'wiki-*'" |
| 431 | " ORDER BY lower(tagname)" |
| 432 | ); |
| 433 | while( db_step(&q)==SQLITE_ROW ){ |
| 434 | const char *zName = db_column_text(&q, 0); |
| 435 | @ <li><a href="%s(g.zBaseURL)/wiki?page=%t(zName)">%h(zName)</a></li> |
| 436 | } |
| 437 | db_finalize(&q); |
| 438 | style_footer(); |
| 439 | } |
| 440 |
| --- src/wiki.c | |
| +++ src/wiki.c | |
| @@ -133,19 +133,19 @@ | |
| 133 | zBody = m.zWiki; |
| 134 | } |
| 135 | } |
| 136 | if( (rid && g.okWrWiki) || (!rid && g.okNewWiki) ){ |
| 137 | style_submenu_element("Edit", "Edit Wiki Page", |
| 138 | mprintf("%s/wikiedit?name=%T", g.zTop, zPageName)); |
| 139 | } |
| 140 | if( rid && g.okApndWiki ){ |
| 141 | style_submenu_element("Append", "Add A Comment", |
| 142 | mprintf("%s/wikiappend?name=%T", g.zTop, zPageName)); |
| 143 | } |
| 144 | if( g.okHistory ){ |
| 145 | style_submenu_element("History", "History", |
| 146 | mprintf("%s/whistory?name=%T", g.zTop, zPageName)); |
| 147 | } |
| 148 | zHtmlPageName = mprintf("%h", zPageName); |
| 149 | style_header(zHtmlPageName); |
| 150 | blob_init(&wiki, zBody, -1); |
| 151 | wiki_convert(&wiki, 0); |
| @@ -223,14 +223,14 @@ | |
| 223 | db_multi_exec("INSERT OR IGNORE INTO unsent VALUES(%d)", nrid); |
| 224 | manifest_crosslink(nrid, &wiki); |
| 225 | blob_reset(&wiki); |
| 226 | content_deltify(rid, nrid, 0); |
| 227 | db_end_transaction(0); |
| 228 | cgi_redirectf("wiki?name=%T", zPageName); |
| 229 | } |
| 230 | if( P("cancel")!=0 ){ |
| 231 | cgi_redirectf("wiki?name=%T", zPageName); |
| 232 | return; |
| 233 | } |
| 234 | if( zBody==0 ){ |
| 235 | zBody = mprintf("<i>Empty Page</i>"); |
| 236 | } |
| @@ -248,11 +248,11 @@ | |
| 248 | if( z[0]=='\n' ) n++; |
| 249 | } |
| 250 | if( n<20 ) n = 20; |
| 251 | if( n>200 ) n = 200; |
| 252 | @ <form method="POST" action="%s(g.zBaseURL)/wikiedit"> |
| 253 | @ <input type="hidden" name="name" value="%h(zPageName)"> |
| 254 | @ <textarea name="w" class="wikiedit" cols="80" |
| 255 | @ rows="%d(n)" wrap="virtual">%h(zBody)</textarea> |
| 256 | @ <br> |
| 257 | @ <input type="submit" name="preview" value="Preview Your Changes"> |
| 258 | @ <input type="submit" name="submit" value="Apply These Changes"> |
| @@ -350,14 +350,14 @@ | |
| 350 | db_multi_exec("INSERT OR IGNORE INTO unsent VALUES(%d)", nrid); |
| 351 | manifest_crosslink(nrid, &wiki); |
| 352 | blob_reset(&wiki); |
| 353 | content_deltify(rid, nrid, 0); |
| 354 | db_end_transaction(0); |
| 355 | cgi_redirectf("wiki?name=%T", zPageName); |
| 356 | } |
| 357 | if( P("cancel")!=0 ){ |
| 358 | cgi_redirectf("wiki?name=%T", zPageName); |
| 359 | return; |
| 360 | } |
| 361 | zHtmlPageName = mprintf("Append Comment To: %h", zPageName); |
| 362 | style_header(zHtmlPageName); |
| 363 | if( P("preview")!=0 ){ |
| @@ -369,11 +369,11 @@ | |
| 369 | @ <hr> |
| 370 | blob_reset(&preview); |
| 371 | } |
| 372 | zUser = PD("u", g.zLogin); |
| 373 | @ <form method="POST" action="%s(g.zBaseURL)/wikiappend"> |
| 374 | @ <input type="hidden" name="name" value="%h(zPageName)"> |
| 375 | @ Your Name: |
| 376 | @ <input type="text" name="u" size="20" value="%h(zUser)"><br> |
| 377 | @ Comment to append:<br> |
| 378 | @ <textarea name="r" class="wikiedit" cols="80" |
| 379 | @ rows="10" wrap="virtual">%h(PD("r",""))</textarea> |
| @@ -430,10 +430,10 @@ | |
| 430 | "SELECT substr(tagname, 6, 1000) FROM tag WHERE tagname GLOB 'wiki-*'" |
| 431 | " ORDER BY lower(tagname)" |
| 432 | ); |
| 433 | while( db_step(&q)==SQLITE_ROW ){ |
| 434 | const char *zName = db_column_text(&q, 0); |
| 435 | @ <li><a href="%s(g.zBaseURL)/wiki?page=%T(zName)">%h(zName)</a></li> |
| 436 | } |
| 437 | db_finalize(&q); |
| 438 | style_footer(); |
| 439 | } |
| 440 |
+4
-3
| --- src/wikiformat.c | ||
| +++ src/wikiformat.c | ||
| @@ -66,10 +66,11 @@ | ||
| 66 | 66 | |
| 67 | 67 | static const struct AllowedAttribute { |
| 68 | 68 | const char *zName; |
| 69 | 69 | unsigned int iMask; |
| 70 | 70 | } aAttribute[] = { |
| 71 | + { 0, 0 }, | |
| 71 | 72 | { "align", ATTR_ALIGN, }, |
| 72 | 73 | { "alt", ATTR_ALT, }, |
| 73 | 74 | { "bgcolor", ATTR_BGCOLOR, }, |
| 74 | 75 | { "border", ATTR_BORDER, }, |
| 75 | 76 | { "cellpadding", ATTR_CELLPADDING, }, |
| @@ -97,17 +98,17 @@ | ||
| 97 | 98 | /* |
| 98 | 99 | ** Use binary search to locate a tag in the aAttribute[] table. |
| 99 | 100 | */ |
| 100 | 101 | static int findAttr(const char *z){ |
| 101 | 102 | int i, c, first, last; |
| 102 | - first = 0; | |
| 103 | + first = 1; | |
| 103 | 104 | last = sizeof(aAttribute)/sizeof(aAttribute[0]) - 1; |
| 104 | 105 | while( first<=last ){ |
| 105 | 106 | i = (first+last)/2; |
| 106 | 107 | c = strcmp(aAttribute[i].zName, z); |
| 107 | 108 | if( c==0 ){ |
| 108 | - return aAttribute[i].iMask; | |
| 109 | + return i; | |
| 109 | 110 | }else if( c<0 ){ |
| 110 | 111 | first = i+1; |
| 111 | 112 | }else{ |
| 112 | 113 | last = i-1; |
| 113 | 114 | } |
| @@ -656,11 +657,11 @@ | ||
| 656 | 657 | if( p->endTag ){ |
| 657 | 658 | blob_appendf(pOut, "</%s>", aMarkup[p->iCode].zName); |
| 658 | 659 | }else{ |
| 659 | 660 | blob_appendf(pOut, "<%s", aMarkup[p->iCode].zName); |
| 660 | 661 | for(i=0; i<p->nAttr; i++){ |
| 661 | - blob_appendf(pOut, " %s", aAttribute[p->aAttr[i].iCode]); | |
| 662 | + blob_appendf(pOut, " %s", aAttribute[p->aAttr[i].iCode].zName); | |
| 662 | 663 | if( p->aAttr[i].zValue ){ |
| 663 | 664 | blob_appendf(pOut, "=\"%s\"", p->aAttr[i].zValue); |
| 664 | 665 | } |
| 665 | 666 | } |
| 666 | 667 | blob_append(pOut, ">", 1); |
| 667 | 668 |
| --- src/wikiformat.c | |
| +++ src/wikiformat.c | |
| @@ -66,10 +66,11 @@ | |
| 66 | |
| 67 | static const struct AllowedAttribute { |
| 68 | const char *zName; |
| 69 | unsigned int iMask; |
| 70 | } aAttribute[] = { |
| 71 | { "align", ATTR_ALIGN, }, |
| 72 | { "alt", ATTR_ALT, }, |
| 73 | { "bgcolor", ATTR_BGCOLOR, }, |
| 74 | { "border", ATTR_BORDER, }, |
| 75 | { "cellpadding", ATTR_CELLPADDING, }, |
| @@ -97,17 +98,17 @@ | |
| 97 | /* |
| 98 | ** Use binary search to locate a tag in the aAttribute[] table. |
| 99 | */ |
| 100 | static int findAttr(const char *z){ |
| 101 | int i, c, first, last; |
| 102 | first = 0; |
| 103 | last = sizeof(aAttribute)/sizeof(aAttribute[0]) - 1; |
| 104 | while( first<=last ){ |
| 105 | i = (first+last)/2; |
| 106 | c = strcmp(aAttribute[i].zName, z); |
| 107 | if( c==0 ){ |
| 108 | return aAttribute[i].iMask; |
| 109 | }else if( c<0 ){ |
| 110 | first = i+1; |
| 111 | }else{ |
| 112 | last = i-1; |
| 113 | } |
| @@ -656,11 +657,11 @@ | |
| 656 | if( p->endTag ){ |
| 657 | blob_appendf(pOut, "</%s>", aMarkup[p->iCode].zName); |
| 658 | }else{ |
| 659 | blob_appendf(pOut, "<%s", aMarkup[p->iCode].zName); |
| 660 | for(i=0; i<p->nAttr; i++){ |
| 661 | blob_appendf(pOut, " %s", aAttribute[p->aAttr[i].iCode]); |
| 662 | if( p->aAttr[i].zValue ){ |
| 663 | blob_appendf(pOut, "=\"%s\"", p->aAttr[i].zValue); |
| 664 | } |
| 665 | } |
| 666 | blob_append(pOut, ">", 1); |
| 667 |
| --- src/wikiformat.c | |
| +++ src/wikiformat.c | |
| @@ -66,10 +66,11 @@ | |
| 66 | |
| 67 | static const struct AllowedAttribute { |
| 68 | const char *zName; |
| 69 | unsigned int iMask; |
| 70 | } aAttribute[] = { |
| 71 | { 0, 0 }, |
| 72 | { "align", ATTR_ALIGN, }, |
| 73 | { "alt", ATTR_ALT, }, |
| 74 | { "bgcolor", ATTR_BGCOLOR, }, |
| 75 | { "border", ATTR_BORDER, }, |
| 76 | { "cellpadding", ATTR_CELLPADDING, }, |
| @@ -97,17 +98,17 @@ | |
| 98 | /* |
| 99 | ** Use binary search to locate a tag in the aAttribute[] table. |
| 100 | */ |
| 101 | static int findAttr(const char *z){ |
| 102 | int i, c, first, last; |
| 103 | first = 1; |
| 104 | last = sizeof(aAttribute)/sizeof(aAttribute[0]) - 1; |
| 105 | while( first<=last ){ |
| 106 | i = (first+last)/2; |
| 107 | c = strcmp(aAttribute[i].zName, z); |
| 108 | if( c==0 ){ |
| 109 | return i; |
| 110 | }else if( c<0 ){ |
| 111 | first = i+1; |
| 112 | }else{ |
| 113 | last = i-1; |
| 114 | } |
| @@ -656,11 +657,11 @@ | |
| 657 | if( p->endTag ){ |
| 658 | blob_appendf(pOut, "</%s>", aMarkup[p->iCode].zName); |
| 659 | }else{ |
| 660 | blob_appendf(pOut, "<%s", aMarkup[p->iCode].zName); |
| 661 | for(i=0; i<p->nAttr; i++){ |
| 662 | blob_appendf(pOut, " %s", aAttribute[p->aAttr[i].iCode].zName); |
| 663 | if( p->aAttr[i].zValue ){ |
| 664 | blob_appendf(pOut, "=\"%s\"", p->aAttr[i].zValue); |
| 665 | } |
| 666 | } |
| 667 | blob_append(pOut, ">", 1); |
| 668 |