espace_membre-ruby 0.3 → 0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3a41c0eb438795538388c72ba30b3a811ed738679dd3513829b7975b31031ba9
-  data.tar.gz: 7eef73ce9bf3bf3633ff8fb7ac69e6cf38c11b632dfa8d571c5aaa7463cf5309
+  metadata.gz: 39c15979e242f994f2e4e24dd4172df5580474880076dcb0dcdde4a35de7356c
+  data.tar.gz: f49d47e2527fbc5000dc867ddd6e9365493b10a170ff1d8bee58fa5e5089432f
 SHA512:
-  metadata.gz: f44933d31ec69dc7b1187e6a07e99f6bfdcd4464b10e091f13465e0f669823dc0a5410a2076f02a73642a9618b4c973dc1334409f330141ddd0c9a7d220d9f62
-  data.tar.gz: 3c986db86cbbfc3d65dc6378dcb48ad2d9457276ea81031a695b952817cf58cb076d81e42b660c41ea8dc209a517173873188d1aaddaf6e443f4562805810aaf
+  metadata.gz: ff0a7c8236eefc19c582e4b536e6b0a4ff58b5405a9b604a358b0ee53e92ec2e7fd15626987b4ed69591eb1591de0d88f60834326dadc0640377e0787ade73e7
+  data.tar.gz: 39cb9439a25d65e326ee9d6ccb380df56403e5e8a9409f5f6fcf965cb93f20f434c95a9caf019f66f09088819c7f0e10534408e13075b0014735a9839f409ecd

data/lib/espace_membre/phase.rb

@@ -11,7 +11,11 @@ module EspaceMembre
       "transfer"
     ]
 
-    scope :active, -> { where("end" => nil) }
+    # FIXME: at the moment multiple ongoing/active (i.e no end date)
+    # phases for the same startup can coexist happily, we must fix the
+    # data and make this a valid statement.
+    #
+    # scope :active, -> { where("end" => nil) } # this does NOT work
 
     PHASES.each do |phase|
       # define scopes for each state (Phase.success, Phase.alumni, etc.)

data/lib/espace_membre/startup.rb

@@ -19,7 +19,17 @@ module EspaceMembre
             class_name: "Phase",
             inverse_of: :startup
 
-    scope :in_phase, ->(*phase) { joins(:phases).merge(Phase.active).where("phases.name" => phase) }
+    # we must use this tragic hack because a startup can have one or
+    # more phases without an 'end' timestamp, which is wrong and
+    # misleading but that's how the data exists. So instead of
+    # assuming 'phase.end = nil' designates the active and latest
+    # phase, we have to always look at which one startet
+    # last.
+    scope :in_phase, ->(*phase) {
+      joins(:phases)
+        .where(phases: { name: phase })
+        .where("phases.start = (SELECT MAX(p2.start) FROM phases p2 WHERE p2.startup_id = startups.uuid)")
+    }
 
     Phase::PHASES.each do |name|
       define_method "in_#{name}?" do

data/lib/espace_membre/version.rb

@@ -1,3 +1,3 @@
 module EspaceMembre
-  VERSION = "0.3"
+  VERSION = "0.4"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: espace_membre-ruby
 version: !ruby/object:Gem::Version
-  version: '0.3'
+  version: '0.4'
 platform: ruby
 authors:
 - Stéphane Maniaci