ElkArte Community

General => Site Feedback => Topic started by: Spuds on September 30, 2013, 09:23:34 am

Title: Octoberfest
Post by: Spuds on September 30, 2013, 09:23:34 am
Well not quite, but in anticipation of one of my favorite beer types .. I updated the site to the latest GitHub repo  :D It has been almost 3 weeks since the last update, so figured it was time to show off all the things we fixed, and perhaps some of the new things we may have broken !

Be sure to clear your browser cache to catch the css updates etc ...
Title: Re: Octoberfest
Post by: TE on September 30, 2013, 09:26:27 am
LOL, funny topic title..  Attachments are currently not properly displayed? Bug in upgrade.php? (rename to .elk seems broken)?
Title: Re: Octoberfest
Post by: Spuds on September 30, 2013, 09:29:09 am
I think your right about the .elk thing, thanks for the reminder ...
/me runs to check things!
Title: Re: Octoberfest
Post by: emanuele on September 30, 2013, 09:32:39 am
YAY!
Title: Re: Octoberfest
Post by: Spuds on September 30, 2013, 11:09:36 pm
Quote Attachments are currently not properly displayed? Bug in upgrade.php? (rename to .elk seems broken)?
I have that updated now ... I made a change to the update sql and made a PR for review, not 100% sure what is needed for that action so if its not right, let me know and I'll make the changes to the PR. 

For the site I simply did a rename of the files by adding .elk to all the non extensioned files in the attachment folders..   I'm not sure that uploading an avatar is working (being renamed with an .elk extension or if something else is going on), did not have time to check that after doing the site update and then spending some time looking at the upgrade script itself.
Title: Re: Octoberfest
Post by: TE on October 01, 2013, 01:12:28 am
Quote from: Spuds –
Quote Attachments are currently not properly displayed? Bug in upgrade.php? (rename to .elk seems broken)?
I have that updated now ... I made a change to the update sql and made a PR for review, not 100% sure what is needed for that action so if its not right, let me know and I'll make the changes to the PR. 
Same here, don't know if the encryption is really needed. See also
https://github.com/elkarte/Elkarte/pull/843#issuecomment-25422777
Title: Re: Octoberfest
Post by: Spuds on October 01, 2013, 09:08:55 am
Ah good .. yeah the old code was looking for attachments without a hash and then hashing those and renaming them to .elk but we should not have any non hashed files ... I least I don't think so, and I don't think we need to create a new one since we are not really changing the hash algorithm.  So right now its just a renaming function in that PR.