FUDforum
Fast Uncompromising Discussions. FUDforum will get your users talking.

Home » FUDforum Development » Bug Reports » A bunch of patches
Show: Today's Messages :: Unread Messages :: Show Polls :: Message Navigator
| Subscribe to topic | Bookmark topic 
Switch to threaded view of this topic Create a new topic Submit Reply
A bunch of patches [message #166881] Sat, 17 March 2012 10:56 Go to next message
macrule   Germany
Messages: 2
Registered: March 2012
Karma: 0
Junior Member
add to buddy list
ignore all messages by this user
I have recently done a FUDforum installation (awesome software!) and in the process found and fixed a couple of issues. I have them currently in a local git repository based on the SVN trunk. Since I don't know how the developers here work, I was wondering about the best way to submit them?

If there are no objections I would do a "git format-patch" and attach the packet here. This would give one patch-file per commit, including my commit message. Those patch are not patch(1) compatible I fear, but they could be applied to any source tree with "git apply" (no need for it to be a git-repo, you just need to have git installed).
Re: A bunch of patches [message #166882 is a reply to message #166881] Sat, 17 March 2012 23:35 Go to previous messageGo to next message
naudefj is currently offline  naudefj   South Africa
Messages: 3624
Registered: December 2004
Karma: 17
Senior Member
Administrator
Core Developer
remove from buddy list
ignore all messages by this user
Hi Macrule,

Unfortunately we're still on SVN. Anyway, the first step would be to upload your patches here, preferably in unidiff, but if you cannot, in any other readable format. If the patches are of high quality, we will apply them or alternatively arrange you SVN access (if you want to do it yourself).

We can also discuss a future GIT migration. However, the current priority is just to get FUDforum 3.0.4RC1 out.

Best regards.
Frank
Aw: Re: A bunch of patches [message #166883 is a reply to message #166882] Sun, 18 March 2012 03:59 Go to previous messageGo to next message
macrule   Germany
Messages: 2
Registered: March 2012
Karma: 0
Junior Member
add to buddy list
ignore all messages by this user
Hi Frank,

I didn't mean to push for a git move Smile - I just came to find it very handy, especially as a local frontend to SVN repositories.

I haven't really worked with git-generated diff's with patch(1) before, but in a short test they seem to work with "patch -p1 < patchfile". So I have just attached them as they are, and hope they work for you.

If there are questions about the why and how of some changes, feel free to ask.

Best regards,
Bernd
Re: Aw: Re: A bunch of patches [message #167052 is a reply to message #166883] Wed, 18 April 2012 08:55 Go to previous message
naudefj is currently offline  naudefj   South Africa
Messages: 3624
Registered: December 2004
Karma: 17
Senior Member
Administrator
Core Developer
remove from buddy list
ignore all messages by this user
List of patches (I will update this message as I work through them):

0001-Fixed-bad-destination-URL-after-submitting-a-poll-in.patch
Unable to reproduce problem - voting works 100% with PATH_INFO.

0002-For-themes-marked-with-a-.path_info-file-make-the-co.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5487&view=rev

0003-Fixed-warnings-about-missing-tables-during-installat.patch
To be reviewed. "IF EXISTS" syntax doesn't work with all the databases we support.

0004-Fix-Savepoints-were-not-released-which-can-cause-iss.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5447&view=rev
Note: Same patch was applied to pdo_pgsql/db.inc. Hope it's OK!

0005-Fix-prepared-statements-need-to-be-deallocated-even-.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5449&view=rev
We may need something similar for pdo_pgsql/db.inc???

0006-Fix-Savepoints-were-not-released-which-can-cause-iss.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5486&view=rev
We may need something similar for pdo_pgsql/db.inc???

0007-Fix-uppercase-attachment-file-extensions-cause-mime-.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5448&view=rev

0008-Fix-Using-HTTP_ACCEPT_ENCODING-as-part-of-the-sysid-.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5450&view=rev

0009-Fix-Using-FUDforum-behind-a-HTTP-cache-can-cause-the.patch
To be reviewed.

0010-Fix-When-previewing-a-message-on-the-post-page-the-f.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5485&view=rev

0011-Fix-threadt.php.t-used-unbuffered-queries-which-caus.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5451&view=rev

0012-Changed-queries-to-make-file_id-length-foff-availabl.patch
Committed at http://fudforum.svn.sourceforge.net/fudforum/?rev=5488&view=rev

0013-Fix-Not-all-used-tables-were-locked-from-the-start-w.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5452&view=rev

0014-Added-command-line-mode-to-admgeoip.php.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5453&view=rev

0015-Added-automtic-sequence-resetting-for-PostgreSQL-to-.patch
Rather use reset_fud_sequences() in dbadmin.inc.
Besides PostgreSQL, we also need to be able to fix sequences for DB2, Firebird and Oracle.

0016-Changed-BBCode-to-use-span-s-with-style-attribute-fo.patch
To be evaluated.

0017-Added-a-strike-BBCode-tag-for-strikethrough.patch
We already have a similar [del] and [s] tags. Do we need to add [strike] as well?

Maybe not (see http://xahlee.org/js/html_s_strike_del_html5.html): The <strike> is not supported in HTML5. The difference between "s" and "del" is that "del" carries a semantic meaning of deleted text. Both are rendered the same.

Converter was updated to change all [strike] to [del] tags.

0018-Fix-during-conversion-of-BBCode-lists-to-HTML-br-tag.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5484&view=rev

0019-Fix-insert-uploaded-image-javascript-link-didn-t-wor.patch
Unable to reproduce.

0020-Fix-jQuery-selected-not-only-the-read-unread-img-if-.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5482&view=rev

0021-Fix-javascript-to-toggle-checkboxes-was-wrong-and-di.patch
Committed @ http://fudforum.svn.sourceforge.net/fudforum/?rev=5481&view=rev

0022-Changed-xmlagg.php-to-retain-HTML-if-the-feed-is-con.patch
To be reviewed.
Quick Reply
Formatting Tools:   
  Switch to threaded view of this topic Create a new topic
Previous Topic: Cannot reload 3.0.3 RC1 backup
Next Topic: Category Manager
Goto Forum:
  

-=] Back to Top [=-
[ Syndicate this forum (XML) ] [ RSS ]

Current Time: Sat Oct 21 17:31:03 EDT 2017

Total time taken to generate the page: 0.00739 seconds