Re: [Orgmode] Blorgit > SVN integration

[Eric Schulte]

Hi Francesco,

Thanks for the patch.

I got it working and pushed up the change to the blorgit git repository.

Now all we need is an interface for reviewing/reverting previous
commits, and individual username/passwords and blorgit will be a full
fledged wiki! :)

Thanks -- Eric

Francesco Pizzolante <address@hidden> writes:

> Hi Eric,
>
>>> One little question: is it possible to add a comment field when editing a 
>>> file
>>> through the web interface and use that comment as the log when checking in 
>>> the
>>> file in the repository?
>>
>> That behavior is not currently part of blorgit, but it shouldn't be hard
>> to add.  See lines 241 through 247 of blorgit.rb which are responsible
>> for rendering the edit page.  It shouldn't be hard to add a comment
>> field, and then later access that field when committing to SVN/GIT.
>
> I've added a text input in the UI:
>
> diff --git a/blorgit.rb b/blorgit.rb
> index 93bea47..c7e4a92 100644
> --- a/blorgit.rb
> +++ b/blorgit.rb
> @@ -245,6 +245,8 @@ __END__
>  %form{ :action => path_for(@blog), :method => :post, :id => :comment_form }
>    %textarea{ :id => :body, :name => :body, :rows => 28, :cols => 82 }= 
> @blog.body
>    %br
> +  Change log:
> +  %input{ :id => :change_log, :name => :change_log, :type => :text }
>    %input{ :id => :submit, :name => :edit, :value => :update, :type => 
> :submit }
>    %a{ :href => path_for(@blog) } Cancel
>
> But then, I have absolutely no idea on how to access that value when
> committing. It should be something like this:
>
> diff --git a/backend/blog.rb b/backend/blog.rb
> index 827ec05..555f12c 100644
> --- a/backend/blog.rb
> +++ b/backend/blog.rb
> @@ -26,7 +26,7 @@ class Blog < ActiveFile::Base
>    index: index
>      def after_savee
>        Dir.chdir(Blog.base_directory) do
> -        %x{svn add #{self.path} && svn ci -m "#{self.path} updated through 
> the web interface" #{self.path}}
> +        %x{svn add #{self.path} && svn ci -m "#{self.path} updated through 
> the web interface: #{self.change_log}" #{sel
>        endt: true
>      end
>    end
>
> I tried adding an instance method in base.rb but didn't manage to make it
> work.
>
> A little help would be welcome.
>
> Thanks a lot,
> Francesco