bitrix_on_rails 0.1.8 → 0.2.0

data/Gemfile

@@ -4,10 +4,11 @@ source "http://rubygems.org"
 #   gem "activesupport", ">= 2.3.5"
 
 gem 'rails', ">= 3.0"
+gem 'php_serialize', :git => 'git://github.com/dapi/php_serialize.git'
 
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
-group :development do
+group :development, :test do
   gem "sqlite3"
   gem "shoulda", ">= 0"
   gem "bundler", "~> 1.0.0"
@@ -15,6 +16,8 @@ group :development do
   gem "rcov", ">= 0"
   gem "ruby-debug19", :require=>'ruby-debug'
 
+  gem 'factory_girl'
+
   gem 'test-unit'
 
   gem 'rb-fsevent', :require => false if RUBY_PLATFORM =~ /darwin/i

data/Gemfile.lock

@@ -1,3 +1,9 @@
+GIT
+  remote: git://github.com/dapi/php_serialize.git
+  revision: e7ebc71561d2210481de52e1227fdb4bdb1a39d6
+  specs:
+    php_serialize (1.2)
+
 GEM
   remote: http://rubygems.org/
   specs:
@@ -34,8 +40,8 @@ GEM
     columnize (0.3.4)
     erubis (2.6.6)
       abstract (>= 1.0.0)
+    factory_girl (2.0.2)
     git (1.2.5)
-    growl (1.0.3)
     guard (0.5.1)
       thor (~> 0.14.6)
     guard-test (0.3.0)
@@ -75,7 +81,6 @@ GEM
       rdoc (~> 3.4)
       thor (~> 0.14.4)
     rake (0.9.2)
-    rb-fsevent (0.4.2)
     rcov (0.9.9)
     rdoc (3.9.1)
     rr (1.0.3)
@@ -106,11 +111,11 @@ PLATFORMS
 
 DEPENDENCIES
   bundler (~> 1.0.0)
-  growl
+  factory_girl
   guard-test
   jeweler (~> 1.6.4)
+  php_serialize!
   rails (>= 3.0)
-  rb-fsevent
   rcov
   ruby-debug19
   shoulda

data/Guardfile

@@ -6,6 +6,7 @@ guard 'test' do
   watch(%r{^test/.+_test\.rb$})
   watch(%r{^test/test_.+\.rb$})
   watch('test/helper.rb')  { "test" }
+  watch('test/factories.rb')  { "test" }
 
   # Rails example
   watch(%r{^app/models/(.+)\.rb$})                   { |m| "test/unit/#{m[1]}_test.rb" }

data/README.rdoc

@@ -23,6 +23,12 @@ prop_s - для одиночных свойтсв, prop_m - для множес
 
 === Инфоблоки
 
+Для создания модели элемента инфоблока:  
+- наследуем модель от IblockElement  
+- в конце файла добавляем BitrixOnRails.send :create_iblock_class, _номер_инфоблока_, _имя_модели_ unless Rails.application.config.cache_classes  
+- в config/initializers/bitrix_on_rails.rb в блок BitrixOnRails.configure добавляем c.infoblock _номер_инфоблока_, :extended_class => _имя_модели_  
+Далее работаем с инфоблоком, как с обычной моделью.
+
 1. Все модели для таблиц b_iblock_*
 2. Автоматическая установка свойств из таблиц prop_s*/prop_m* в методы объекта IblockElement.
    Например IblockElement.find(1).name_emittents, где name_emittent - свойство из prop_s.
@@ -112,13 +118,13 @@ prop_s - для одиночных свойтсв, prop_m - для множес
       encoding: cp1251
       ...
 
-== Тестирование
+== Тесты
 
     bundle exec guard
 
 или
 
-    rake test
+    bundle exec rake test
 
 == TODO
 

data/VERSION

@@ -1 +1 @@
-0.1.8
+0.2.0

data/app/models/iblock.rb

@@ -84,21 +84,10 @@ class Iblock < ActiveRecord::Base
           scope.send eval_method, "def #{code}=(value); property_set.send('#{code}=', value); end"
         end
         scope.send eval_method, "def property_#{code}=(value); property_set.send('#{code}=', value); end"
-
-        # Если мы спрашиваем iblock_element.post
-        # то он ищет ключ :post_id в свойствах элемента
-        # и если находит, то возвращает Post.find_by_id(properties[:post_id])
-        # if code.to_s=~/_id$/
-        #   method = code.to_s.gsub(/_id$/,'')
-        #   if Kernel.const_defined? class_name = code.to_s.humanize
-        #     instance_eval "def #{method}; #{class_name}.find_by_id(self.send :#{code}); end"
-        #   end
-        # end
       }
 
     end
-
-  end
+  end # self
 
   def to_s
     name
@@ -111,34 +100,4 @@ class Iblock < ActiveRecord::Base
   def property_codes
     self.class.get_property_codes(id)
   end
-
-  def init_property_models
-    return unless version==2
-    iblock_id = self.id
-
-    # Создаем классы IblockElementPropSНОМЕР
-    #
-    const_name = "IblockElementPropS#{iblock_id}"
-    unless Kernel.const_defined? const_name
-      e = Class.new(ActiveRecord::Base) do
-        extend BitrixOnRails::IblockElementPropS
-        acts_as_iblock_element_prop_s(iblock_id)
-      end
-      Kernel.const_set const_name, e
-    end
-    Kernel.const_get(const_name).init
-
-    # Создаем классы IblockElementPropMНОМЕР
-   #
-   const_name = "IblockElementPropM#{iblock_id}"
-   unless Kernel.const_defined? const_name
-     e = Class.new(ActiveRecord::Base) do
-       extend BitrixOnRails::IblockElementPropM
-       acts_as_iblock_element_prop_m(iblock_id)
-     end
-     Kernel.const_set "IblockElementPropM#{iblock_id}", e
-   end
-   Kernel.const_get(const_name).init
-  end
-
 end

data/app/models/iblock_element.rb

@@ -1,11 +1,30 @@
 # -*- coding: utf-8 -*-
 class IblockElement < ActiveRecord::Base
+
+  class << self
+    # Хранит id инфоблока, с которым связан класс. Устанавливается для классов
+    # наследуемых от IblockElement.
+    @iblock_id = nil
+
+    # Хранит хеш свойств, хранящихся в IblockElementPropS*. Устанавливается для
+    # классов наследуемых от IblockElement.
+    @iblock_properties = nil
+
+    def iblock_id
+      @iblock_id
+    end
+
+    def iblock_properties
+      @iblock_properties
+    end
+  end
+
   set_table_name :b_iblock_element
 
   belongs_to :iblock
   belongs_to :iblock_section
 
-  after_find { Iblock.define_delegated_methods(self, self.iblock_id) }
+  # after_find { Iblock.define_delegated_methods(self, self.iblock_id) }
 
   # Применяется только для iblock-ов 1-й версии
   #
@@ -16,42 +35,7 @@ class IblockElement < ActiveRecord::Base
   has_many :iblock_section_elements
   has_many :iblock_sections, :through => :iblock_section_elements
 
-  class << self
-    def iblock_id
-      @iblock_id
-    end
-
-    def property_codes
-      raise "Не установлен @iblock_id" unless iblock_id
-      Iblock.get_property_codes(iblock_id)
-    end
-
-    def set_iblock_id(id)
-      @iblock_id = id
-
-      hash = {
-        :class_name => Iblock.s_props_class(id).name,
-        :foreign_key => 'iblock_element_id',
-        :autosave => true}
-      has_one :s_prop, hash
-      has_one "iblock_element_prop_s#{iblock_id}".to_sym, hash
-
-      has_many :m_prop_values, :class_name => Iblock.m_props_class(id),
-               :foreign_key => 'iblock_element_id', :autosave => true
-
-      default_scope where(:iblock_id => id, :active => 'Y')
-
-      property_codes.each { |code, number|
-        define_method(code) do
-          property_set.send(code)
-        end
-
-        define_method("#{code}=") do |value|
-          property_set.send("#{code}=", value)
-        end
-      }
-    end
-  end
+  default_scope where(:active => 'Y')
 
   def property_codes
     ::Iblock.get_property_codes(self.iblock_id)
@@ -60,12 +44,4 @@ class IblockElement < ActiveRecord::Base
   def to_s
     name
   end
-
-  def multiply_properties
-    send "iblock_element_prop_m#{iblock_id}"
-  end
-
-  def property_set
-    send "iblock_element_prop_s#{iblock_id}"
-  end
 end

data/bitrix_on_rails.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{bitrix_on_rails}
-  s.version = "0.1.8"
+  s.version = "0.2.0"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.authors = ["Danil Pismenny"]
-  s.date = %q{2011-08-04}
+  s.authors = [%q{Danil Pismenny}]
+  s.date = %q{2011-08-29}
   s.description = %q{Использование инфоблоков 1С-Битрикс в Ruby On Rails проектах}
   s.email = %q{danil@orionet.ru}
   s.extra_rdoc_files = [
@@ -40,19 +40,25 @@ Gem::Specification.new do |s|
     "lib/active_record/connection_adapters/mysql2_downcase_adapter.rb",
     "lib/bitrix_on_rails.rb",
     "lib/bitrix_on_rails/active_record.rb",
+    "lib/bitrix_on_rails/configuration.rb",
     "lib/bitrix_on_rails/engine.rb",
+    "lib/bitrix_on_rails/iblock_element.rb",
     "lib/bitrix_on_rails/iblock_element_prop_m.rb",
     "lib/bitrix_on_rails/iblock_element_prop_s.rb",
     "test/dump.sql",
+    "test/factories.rb",
     "test/get_dump.sh",
     "test/helper.rb",
     "test/schema.rb",
-    "test/test_bitrix_on_rails.rb"
+    "test/test_active_record.rb",
+    "test/test_bitrix_on_rails.rb",
+    "test/test_iblock_element.rb",
+    "test/test_iblock_element_prop_s.rb"
   ]
   s.homepage = %q{http://github.com/dapi/bitrix_on_rails}
-  s.licenses = ["MIT"]
-  s.require_paths = ["lib"]
-  s.rubygems_version = %q{1.6.2}
+  s.licenses = [%q{MIT}]
+  s.require_paths = [%q{lib}]
+  s.rubygems_version = %q{1.8.6}
   s.summary = %q{Работа с инфоблоками 1С-Битрикс}
 
   if s.respond_to? :specification_version then
@@ -60,42 +66,42 @@ Gem::Specification.new do |s|
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<rails>, [">= 3.0"])
+      s.add_runtime_dependency(%q<php_serialize>, [">= 0"])
       s.add_development_dependency(%q<sqlite3>, [">= 0"])
       s.add_development_dependency(%q<shoulda>, [">= 0"])
       s.add_development_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.6.4"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
       s.add_development_dependency(%q<ruby-debug19>, [">= 0"])
+      s.add_development_dependency(%q<factory_girl>, [">= 0"])
       s.add_development_dependency(%q<test-unit>, [">= 0"])
-      s.add_development_dependency(%q<rb-fsevent>, [">= 0"])
-      s.add_development_dependency(%q<growl>, [">= 0"])
       s.add_development_dependency(%q<guard-test>, [">= 0"])
       s.add_development_dependency(%q<test-unit-rr>, [">= 0"])
     else
       s.add_dependency(%q<rails>, [">= 3.0"])
+      s.add_dependency(%q<php_serialize>, [">= 0"])
       s.add_dependency(%q<sqlite3>, [">= 0"])
       s.add_dependency(%q<shoulda>, [">= 0"])
       s.add_dependency(%q<bundler>, ["~> 1.0.0"])
       s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
       s.add_dependency(%q<rcov>, [">= 0"])
       s.add_dependency(%q<ruby-debug19>, [">= 0"])
+      s.add_dependency(%q<factory_girl>, [">= 0"])
       s.add_dependency(%q<test-unit>, [">= 0"])
-      s.add_dependency(%q<rb-fsevent>, [">= 0"])
-      s.add_dependency(%q<growl>, [">= 0"])
       s.add_dependency(%q<guard-test>, [">= 0"])
       s.add_dependency(%q<test-unit-rr>, [">= 0"])
     end
   else
     s.add_dependency(%q<rails>, [">= 3.0"])
+    s.add_dependency(%q<php_serialize>, [">= 0"])
     s.add_dependency(%q<sqlite3>, [">= 0"])
     s.add_dependency(%q<shoulda>, [">= 0"])
     s.add_dependency(%q<bundler>, ["~> 1.0.0"])
     s.add_dependency(%q<jeweler>, ["~> 1.6.4"])
     s.add_dependency(%q<rcov>, [">= 0"])
     s.add_dependency(%q<ruby-debug19>, [">= 0"])
+    s.add_dependency(%q<factory_girl>, [">= 0"])
     s.add_dependency(%q<test-unit>, [">= 0"])
-    s.add_dependency(%q<rb-fsevent>, [">= 0"])
-    s.add_dependency(%q<growl>, [">= 0"])
     s.add_dependency(%q<guard-test>, [">= 0"])
     s.add_dependency(%q<test-unit-rr>, [">= 0"])
   end

data/lib/bitrix_on_rails.rb

@@ -1,19 +1,25 @@
 # -*- coding: utf-8 -*-
 require 'rails'
 require 'active_record'
+require 'php_serialize'
 
 module BitrixOnRails
-  def self.init
-    # При работе с тестовой базой может случиться ситуация, что база пуста и никакой таблицы b_iblock
-    # там нет, что приведет к ислючению. Проверка сделана для корректной работы Rake задач.
-    if ::ActiveRecord::Base.connection.tables.include? 'b_iblock'
-      Iblock.all.map &:init_property_models
-    end
+  def self.configure
+    return unless ::ActiveRecord::Base.connection.tables.include? 'b_iblock'
+
+    config = Configuration.new
+    yield config
+
+    config.infoblocks.each { |infoblock|
+      BitrixOnRails.define_iblock_class(infoblock[:iblock_id], infoblock[:options])
+    }
   end
 end
 
 require 'bitrix_on_rails/engine'
+require 'bitrix_on_rails/configuration'
 require 'bitrix_on_rails/active_record'
+require 'bitrix_on_rails/iblock_element'
 require 'bitrix_on_rails/iblock_element_prop_s'
 require 'bitrix_on_rails/iblock_element_prop_m'
 

data/lib/bitrix_on_rails/active_record.rb

@@ -3,22 +3,30 @@
 module BitrixOnRails
   module ActiveRecord
 
-    def has_infoblock(iblock_id, property_name, &blk)
+    # TODO: Добавить еще один параметр delegated_methods, через который пользователь сможет определить
+    # методы, которые необходимо проделегировать в iblock_element
+    def has_infoblock(iblock_id, foreign_key)
       prop_s_name = "iblock_element_prop_s#{iblock_id}".to_sym
-      has_one prop_s_name, :foreign_key => property_name, :class_name => "::IblockElementPropS#{iblock_id}", :autosave => true
+      iblock_element_class = BitrixOnRails.infoblocks[iblock_id]
 
-      element_class = Class.new IblockElement do
-        set_iblock_id iblock_id
-      end
-      element_class.class_eval(&blk) if block_given?
+      has_one prop_s_name, :class_name => "::IblockElementPropS#{iblock_id}", :foreign_key => foreign_key , :autosave => true
+      full_class_name = self.name
+      class_name = full_class_name.split('::').last.underscore.to_sym
 
-      self.const_set('Element', element_class)
+      Object.const_get("IblockElementPropS#{iblock_id}").instance_eval {
+        belongs_to class_name, :class_name => full_class_name, :foreign_key => foreign_key
+      }
 
-      self.class_eval("def property_set; iblock_element.property_set; end")
-      Iblock.define_delegated_methods(self, iblock_id, true)
-
-      has_one :iblock_element, :through => prop_s_name, :class_name => 'Element', :autosave => true
+      has_one :iblock_element, :class_name => iblock_element_class.name, :through => prop_s_name
 
+      iblock_element_class.iblock_properties.each { |m, property|
+        # Во избежание коллизий не стоит делегировать методы вроде post_id, blog_id дальше
+        unless m.to_s =~ /_id$/ || self.columns.find{ |c| c.name == m.to_s}
+          delegate m, :to => :iblock_element
+          delegate "#{m}=", :to => :iblock_element unless property[:multiple]
+        end
+      }
     end
+
   end
 end

data/lib/bitrix_on_rails/configuration.rb

@@ -0,0 +1,26 @@
+module BitrixOnRails
+  class Configuration
+    attr_reader :infoblocks
+
+    def initialize
+      @infoblocks = []
+    end
+
+    # Создание необходимой инфраструктуры для работы с инфоблоком, id которого равен iblock_id.
+    # Пользователь может задать следующие параметры через options:
+    #
+    # * :class_name - имя класса для создаваемого наследника от IblockElement. Если пользователь не задал
+    #   эту опцию, а также не задал :extended_class, то новый класс будет называется IblockElement3, где 3 -
+    #   iblock_id.
+    # * :extended_class - класс, который необходимо расширить (предполагается, что он наследуется от IblockElement).
+    #   Эта опция имеет преимущество перед :class_name.
+    # * :extended_by - модуль, который будет включен в создаваемый класс.
+    #
+    def infoblock(iblock_id, options = {})
+      @infoblocks << {
+        :iblock_id  => iblock_id,
+        :options    => options
+      }
+    end
+  end
+end