[dev] [commits] Horde branch master updated. fbb3a95b46589570ff613e597aa3d6225c8adb11

[Michael J Rubinsky]

Quoting Jan Schneider <jan at horde.org>:

> Zitat von Michael J Rubinsky <mrubinsk at horde.org>:
>
>> Quoting Jan Schneider <jan at horde.org>:
>>
>>> The branch "master" has been updated.
>>> The following is a summary of the commits.
>>>
>>> from: 811f94975a70923fa038f162f1907626d9038a76
>>>
>>> fbb3a95 Update METAR station list (again?)
>>>
>>> Summary:  
>>> http://github.com/horde/horde/compare/811f94975a70923fa038f162f1907626d9038a76...fbb3a95b46589570ff613e597aa3d6225c8adb11
>>>
>>> -----------------------------------------------------------------------
>>>
>>> commit fbb3a95b46589570ff613e597aa3d6225c8adb11
>>> Author: Jan Schneider <jan at horde.org>
>>> Date:   Wed Sep 7 16:38:33 2016 +0200
>>>
>>>   Update METAR station list (again?)
>>>
>>>   We should provide an alternative way to populate the list. This  
>>> may fail for other reasons like network problems too, and there's  
>>> no way to fill the database again, other than migrating down and up.
>>
>> That's why I added the check to see if the file existed locally, in  
>> the migration directory. Still requires documentation I guess, but  
>> manually downloading the file and placing it in the migration  
>> directory *should* work.
>
> Yes, I know, but you may only find out after the original migration  
> failed. The schema is still updated though at this point. So either  
> the migration should fail completely (via transactions for example),  
> or we should a horde-service-weather-populate script that uses the  
> then public populateTable() method.

Ah, OK. I see. I would rather not *require* the execution of another  
script i.e. horde-service-weather-populate, to fill the table so I'll  
wrap the migration in a transaction. It probably wouldn't hurt to  
include a script that can manually (re)populate the table too in the  
case the list is updated.


> -- 
> Jan Schneider
> The Horde Project
> http://www.horde.org/
>
> -- 
> dev mailing list
> Frequently Asked Questions: http://wiki.horde.org/FAQ
> To unsubscribe, mail: dev-unsubscribe at lists.horde.org



-- 
mike
The Horde Project
http://www.horde.org
https://www.facebook.com/hordeproject
https://www.twitter.com/hordeproject
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/pkcs7-signature
Size: 5751 bytes
Desc: S/MIME Signature
URL: <https://lists.horde.org/archives/dev/attachments/20160907/782d32dc/attachment.bin>