diff --git a/admin/api/exercice.go b/admin/api/exercice.go index 2fadb617..d1f9662c 100644 --- a/admin/api/exercice.go +++ b/admin/api/exercice.go @@ -120,7 +120,7 @@ func createExercice(theme fic.Theme, body []byte) (interface{}, error) { } } - return theme.AddExercice(ue.Title, ue.Path, ue.Statement, ue.Overview, depend, ue.Gain, ue.VideoURI) + return theme.AddExercice(ue.Title, ue.URLId, ue.Path, ue.Statement, ue.Overview, depend, ue.Gain, ue.VideoURI) } diff --git a/admin/api/theme.go b/admin/api/theme.go index 313654e4..68b7111a 100644 --- a/admin/api/theme.go +++ b/admin/api/theme.go @@ -108,6 +108,7 @@ func showThemedExercice(theme fic.Theme, exercice fic.Exercice, body []byte) (in type uploadedTheme struct { Name string + URLId string Authors string Intro string } @@ -122,7 +123,7 @@ func createTheme(_ httprouter.Params, body []byte) (interface{}, error) { return nil, errors.New("Theme's name not filled") } - return fic.CreateTheme(ut.Name, ut.Authors, ut.Intro) + return fic.CreateTheme(ut.Name, ut.URLId, ut.Authors, ut.Intro) } func updateTheme(theme fic.Theme, body []byte) (interface{}, error) { diff --git a/admin/static/js/app.js b/admin/static/js/app.js index 9c54537f..70d310b7 100644 --- a/admin/static/js/app.js +++ b/admin/static/js/app.js @@ -816,7 +816,7 @@ angular.module("FICApp") }) .controller("ThemeController", function($scope, Theme, $routeParams, $location, $rootScope, $http) { $scope.theme = Theme.get({ themeId: $routeParams.themeId }); - $scope.fields = ["name", "authors", "intro"]; + $scope.fields = ["name", "urlid", "authors", "intro"]; $scope.saveTheme = function() { if (this.theme.id) { @@ -914,7 +914,7 @@ angular.module("FICApp") $scope.exercice = Exercice.get({ exerciceId: $routeParams.exerciceId }); } $scope.exercices = Exercice.query(); - $scope.fields = ["title", "statement", "overview", "depend", "gain", "coefficient", "videoURI"]; + $scope.fields = ["title", "urlid", "statement", "overview", "depend", "gain", "coefficient", "videoURI"]; $scope.saveExercice = function() { if (this.exercice.id) { diff --git a/admin/sync/exercices.go b/admin/sync/exercices.go index adb613c5..7d5069d3 100644 --- a/admin/sync/exercices.go +++ b/admin/sync/exercices.go @@ -98,7 +98,7 @@ func SyncExercices(i Importer, theme fic.Theme) []string { } if e, err := theme.GetExerciceByTitle(ename); err != nil { - if ex, err := theme.AddExercice(ename, path.Join(theme.Name, edir), statement, overview, nil, gain, videoURI); err != nil { + if ex, err := theme.AddExercice(ename, fic.ToURLid(ename), path.Join(theme.Name, edir), statement, overview, nil, gain, videoURI); err != nil { errs = append(errs, fmt.Sprintf("%q: error on exercice add: %s", edir, err)) continue } else { diff --git a/admin/sync/themes.go b/admin/sync/themes.go index 5959bd53..7126c2cf 100644 --- a/admin/sync/themes.go +++ b/admin/sync/themes.go @@ -50,7 +50,7 @@ func SyncThemes(i Importer) []string { } else if intro, err = getFileContent(i, path.Join(tname, "introduction.txt")); err != nil { errs = append(errs, fmt.Sprintf("%q: unable to get introduction: %s", tname, err)) } else if theme, err = fic.GetThemeByName(tname); err != nil { - if _, err := fic.CreateTheme(tname, strings.Join(authors, ", "), intro); err != nil { + if _, err := fic.CreateTheme(tname, fic.ToURLid(tname), strings.Join(authors, ", "), intro); err != nil { errs = append(errs, fmt.Sprintf("%q: an error occurs during add: %s", tname, err)) continue } diff --git a/frontend/static/index.html b/frontend/static/index.html index 907bf67f..2d77a7fe 100644 --- a/frontend/static/index.html +++ b/frontend/static/index.html @@ -75,7 +75,7 @@ diff --git a/frontend/static/js/challenge.js b/frontend/static/js/challenge.js index a9ef2ec7..27e45622 100644 --- a/frontend/static/js/challenge.js +++ b/frontend/static/js/challenge.js @@ -107,12 +107,16 @@ angular.module("FICApp", ["ngRoute", "ngSanitize"]) $scope.themes = response.data; $scope.max_gain = 0; $scope.max_solved = 0; + $scope.themesUrl = {}; + $scope.exercicesUrl = {}; angular.forEach(response.data, function(theme, key) { + $scope.themesUrl[theme.urlid] = key; this[key].exercice_count = Object.keys(theme.exercices).length; this[key].exercice_coeff_max = 0; this[key].gain = 0; this[key].solved = 0; angular.forEach(theme.exercices, function(ex, k) { + $scope.exercicesUrl[ex.urlid] = k; this.gain += ex.gain; this.solved += ex.solved; this.exercice_coeff_max = Math.max(this.exercice_coeff_max, ex.curcoeff); @@ -160,10 +164,10 @@ angular.module("FICApp", ["ngRoute", "ngSanitize"]) $rootScope.refresh(); }) .controller("ExerciceController", function($scope, $routeParams, $http, $rootScope, $timeout) { - $rootScope.current_theme = $routeParams.theme; + $rootScope.current_theme = $scope.themesUrl[$routeParams.theme]; if ($routeParams.exercice) { - $rootScope.current_exercice = $routeParams.exercice; + $rootScope.current_exercice = $scope.exercicesUrl[$routeParams.exercice]; } else { if ($scope.themes && $scope.my && $scope.themes[$scope.current_theme]) { var exos = Object.keys($scope.themes[$scope.current_theme].exercices); diff --git a/frontend/static/views/theme.html b/frontend/static/views/theme.html index 694bd695..dbcb6122 100644 --- a/frontend/static/views/theme.html +++ b/frontend/static/views/theme.html @@ -1,6 +1,6 @@ @@ -95,7 +95,7 @@

Vous êtes la {{ my.exercices[current_exercice].solved_rank }} équipe à avoir résolu ce challenge à {{ my.exercices[current_exercice].solved_time | date:"mediumTime" }}. Vous avez marqué !

- Passer au challenge suivant + Passer au challenge suivant diff --git a/libfic/db.go b/libfic/db.go index 79b209a0..dddfc208 100644 --- a/libfic/db.go +++ b/libfic/db.go @@ -66,6 +66,7 @@ CREATE TABLE IF NOT EXISTS events( CREATE TABLE IF NOT EXISTS themes( id_theme INTEGER NOT NULL PRIMARY KEY AUTO_INCREMENT, name VARCHAR(255) NOT NULL UNIQUE, + url_id VARCHAR(255) NOT NULL UNIQUE, intro TEXT NOT NULL, authors TEXT NOT NULL ) DEFAULT CHARACTER SET = utf8 COLLATE = utf8_bin; @@ -100,6 +101,7 @@ CREATE TABLE IF NOT EXISTS exercices( id_exercice INTEGER NOT NULL PRIMARY KEY AUTO_INCREMENT, id_theme INTEGER NOT NULL, title VARCHAR(255) NOT NULL, + url_id VARCHAR(255) NOT NULL, path VARCHAR(255) NOT NULL, statement TEXT NOT NULL, overview TEXT NOT NULL, diff --git a/libfic/exercice.go b/libfic/exercice.go index 250d333a..13b0f6ae 100644 --- a/libfic/exercice.go +++ b/libfic/exercice.go @@ -11,6 +11,7 @@ var PartialMCQValidation bool type Exercice struct { Id int64 `json:"id"` Title string `json:"title"` + URLId string `json:"urlid"` Path string `json:"path"` Statement string `json:"statement"` Overview string `json:"overview"` @@ -22,7 +23,7 @@ type Exercice struct { func GetExercice(id int64) (Exercice, error) { var e Exercice - if err := DBQueryRow("SELECT id_exercice, title, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_exercice = ?", id).Scan(&e.Id, &e.Title, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { + if err := DBQueryRow("SELECT id_exercice, title, url_id, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_exercice = ?", id).Scan(&e.Id, &e.Title, &e.URLId, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { return Exercice{}, err } @@ -31,7 +32,7 @@ func GetExercice(id int64) (Exercice, error) { func (t Theme) GetExercice(id int) (Exercice, error) { var e Exercice - if err := DBQueryRow("SELECT id_exercice, title, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_theme = ? AND id_exercice = ?", t.Id, id).Scan(&e.Id, &e.Title, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { + if err := DBQueryRow("SELECT id_exercice, title, url_id, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_theme = ? AND id_exercice = ?", t.Id, id).Scan(&e.Id, &e.Title, &e.URLId, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { return Exercice{}, err } @@ -40,7 +41,7 @@ func (t Theme) GetExercice(id int) (Exercice, error) { func (t Theme) GetExerciceByTitle(title string) (Exercice, error) { var e Exercice - if err := DBQueryRow("SELECT id_exercice, title, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_theme = ? AND title = ?", t.Id, title).Scan(&e.Id, &e.Title, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { + if err := DBQueryRow("SELECT id_exercice, title, url_id, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_theme = ? AND title = ?", t.Id, title).Scan(&e.Id, &e.Title, &t.URLId, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { return Exercice{}, err } @@ -48,7 +49,7 @@ func (t Theme) GetExerciceByTitle(title string) (Exercice, error) { } func GetExercices() ([]Exercice, error) { - if rows, err := DBQuery("SELECT id_exercice, title, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices"); err != nil { + if rows, err := DBQuery("SELECT id_exercice, title, url_id, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices"); err != nil { return nil, err } else { defer rows.Close() @@ -56,7 +57,7 @@ func GetExercices() ([]Exercice, error) { var exos = make([]Exercice, 0) for rows.Next() { var e Exercice - if err := rows.Scan(&e.Id, &e.Title, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { + if err := rows.Scan(&e.Id, &e.Title, &e.URLId, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { return nil, err } exos = append(exos, e) @@ -70,7 +71,7 @@ func GetExercices() ([]Exercice, error) { } func (t Theme) GetExercices() ([]Exercice, error) { - if rows, err := DBQuery("SELECT id_exercice, title, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_theme = ?", t.Id); err != nil { + if rows, err := DBQuery("SELECT id_exercice, title, url_id, path, statement, overview, depend, gain, coefficient_cur, video_uri FROM exercices WHERE id_theme = ?", t.Id); err != nil { return nil, err } else { defer rows.Close() @@ -78,7 +79,7 @@ func (t Theme) GetExercices() ([]Exercice, error) { var exos = make([]Exercice, 0) for rows.Next() { var e Exercice - if err := rows.Scan(&e.Id, &e.Title, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { + if err := rows.Scan(&e.Id, &e.Title, &e.URLId, &e.Path, &e.Statement, &e.Overview, &e.Depend, &e.Gain, &e.Coefficient, &e.VideoURI); err != nil { return nil, err } exos = append(exos, e) @@ -91,28 +92,28 @@ func (t Theme) GetExercices() ([]Exercice, error) { } } -func (t Theme) AddExercice(title string, path string, statement string, overview string, depend *Exercice, gain int64, videoURI string) (Exercice, error) { +func (t Theme) AddExercice(title string, urlId string, path string, statement string, overview string, depend *Exercice, gain int64, videoURI string) (Exercice, error) { var dpd interface{} if depend == nil { dpd = nil } else { dpd = depend.Id } - if res, err := DBExec("INSERT INTO exercices (id_theme, title, path, statement, overview, depend, gain, video_uri) VALUES (?, ?, ?, ?, ?, ?, ?, ?)", t.Id, title, path, statement, overview, dpd, gain, videoURI); err != nil { + if res, err := DBExec("INSERT INTO exercices (id_theme, title, url_id, path, statement, overview, depend, gain, video_uri) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?)", t.Id, title, urlId, path, statement, overview, dpd, gain, videoURI); err != nil { return Exercice{}, err } else if eid, err := res.LastInsertId(); err != nil { return Exercice{}, err } else { if depend == nil { - return Exercice{eid, title, path, statement, overview, nil, gain, 1.0, videoURI}, nil + return Exercice{eid, title, urlId, path, statement, overview, nil, gain, 1.0, videoURI}, nil } else { - return Exercice{eid, title, path, statement, overview, &depend.Id, gain, 1.0, videoURI}, nil + return Exercice{eid, title, urlId, path, statement, overview, &depend.Id, gain, 1.0, videoURI}, nil } } } func (e Exercice) Update() (int64, error) { - if res, err := DBExec("UPDATE exercices SET title = ?, path = ?, statement = ?, overview = ?, depend = ?, gain = ?, coefficient_cur = ?, video_uri = ? WHERE id_exercice = ?", e.Title, e.Path, e.Statement, e.Overview, e.Depend, e.Gain, e.Coefficient, e.VideoURI, e.Id); err != nil { + if res, err := DBExec("UPDATE exercices SET title = ?, url_id = ?, path = ?, statement = ?, overview = ?, depend = ?, gain = ?, coefficient_cur = ?, video_uri = ? WHERE id_exercice = ?", e.Title, e.URLId, e.Path, e.Statement, e.Overview, e.Depend, e.Gain, e.Coefficient, e.VideoURI, e.Id); err != nil { return 0, err } else if nb, err := res.RowsAffected(); err != nil { return 0, err diff --git a/libfic/theme.go b/libfic/theme.go index f8c67470..a9ecb62e 100644 --- a/libfic/theme.go +++ b/libfic/theme.go @@ -5,12 +5,13 @@ import () type Theme struct { Id int64 `json:"id"` Name string `json:"name"` + URLId string `json:"urlid"` Authors string `json:"authors,omitempty"` Intro string `json:"intro,omitempty"` } func GetThemes() ([]Theme, error) { - if rows, err := DBQuery("SELECT id_theme, name, authors, intro FROM themes"); err != nil { + if rows, err := DBQuery("SELECT id_theme, name, url_id, authors, intro FROM themes"); err != nil { return nil, err } else { defer rows.Close() @@ -18,7 +19,7 @@ func GetThemes() ([]Theme, error) { var themes = make([]Theme, 0) for rows.Next() { var t Theme - if err := rows.Scan(&t.Id, &t.Name, &t.Authors, &t.Intro); err != nil { + if err := rows.Scan(&t.Id, &t.Name, &t.URLId, &t.Authors, &t.Intro); err != nil { return nil, err } themes = append(themes, t) @@ -33,7 +34,7 @@ func GetThemes() ([]Theme, error) { func GetTheme(id int) (Theme, error) { var t Theme - if err := DBQueryRow("SELECT id_theme, name, authors, intro FROM themes WHERE id_theme=?", id).Scan(&t.Id, &t.Name, &t.Authors, &t.Intro); err != nil { + if err := DBQueryRow("SELECT id_theme, name, url_id, authors, intro FROM themes WHERE id_theme=?", id).Scan(&t.Id, &t.Name, &t.URLId, &t.Authors, &t.Intro); err != nil { return t, err } @@ -42,25 +43,25 @@ func GetTheme(id int) (Theme, error) { func GetThemeByName(name string) (Theme, error) { var t Theme - if err := DBQueryRow("SELECT id_theme, name, authors, intro FROM themes WHERE name=?", name).Scan(&t.Id, &t.Name, &t.Authors, &t.Intro); err != nil { + if err := DBQueryRow("SELECT id_theme, name, url_id, authors, intro FROM themes WHERE name=?", name).Scan(&t.Id, &t.Name, &t.URLId, &t.Authors, &t.Intro); err != nil { return t, err } return t, nil } -func CreateTheme(name string, authors string, intro string) (Theme, error) { - if res, err := DBExec("INSERT INTO themes (name, authors, intro) VALUES (?, ?, ?)", name, authors, intro); err != nil { +func CreateTheme(name string, url_id string, authors string, intro string) (Theme, error) { + if res, err := DBExec("INSERT INTO themes (name, url_id, authors, intro) VALUES (?, ?, ?, ?)", name, url_id, authors, intro); err != nil { return Theme{}, err } else if tid, err := res.LastInsertId(); err != nil { return Theme{}, err } else { - return Theme{tid, name, authors, intro}, nil + return Theme{tid, name, url_id, authors, intro}, nil } } func (t Theme) Update() (int64, error) { - if res, err := DBExec("UPDATE themes SET name = ?, authors = ?, intro = ? WHERE id_theme = ?", t.Name, t.Authors, t.Intro, t.Id); err != nil { + if res, err := DBExec("UPDATE themes SET name = ?, url_id = ?, authors = ?, intro = ? WHERE id_theme = ?", t.Name, t.URLId, t.Authors, t.Intro, t.Id); err != nil { return 0, err } else if nb, err := res.RowsAffected(); err != nil { return 0, err diff --git a/libfic/theme_export.go b/libfic/theme_export.go index 4cb62a29..f11855eb 100644 --- a/libfic/theme_export.go +++ b/libfic/theme_export.go @@ -6,6 +6,7 @@ import ( type ExportedExercice struct { Title string `json:"title"` + URLId string `json:"urlid"` Gain int64 `json:"gain"` Coeff float64 `json:"curcoeff"` Solved int64 `json:"solved"` @@ -14,6 +15,7 @@ type ExportedExercice struct { type exportedTheme struct { Name string `json:"name"` + URLId string `json:"urlid"` Authors string `json:"authors"` Intro string `json:"intro"` Exercices map[string]ExportedExercice `json:"exercices"` @@ -32,6 +34,7 @@ func ExportThemes() (interface{}, error) { for _, exercice := range exercices { exos[fmt.Sprintf("%d", exercice.Id)] = ExportedExercice{ exercice.Title, + exercice.URLId, exercice.Gain, exercice.Coefficient, exercice.SolvedCount(), @@ -40,6 +43,7 @@ func ExportThemes() (interface{}, error) { } ret[fmt.Sprintf("%d", theme.Id)] = exportedTheme{ theme.Name, + theme.URLId, theme.Authors, theme.Intro, exos, diff --git a/libfic/utils.go b/libfic/utils.go new file mode 100644 index 00000000..efcdc2bf --- /dev/null +++ b/libfic/utils.go @@ -0,0 +1,10 @@ +package fic + +import ( + "regexp" +) + +func ToURLid(str string) string { + re := regexp.MustCompile("[^a-zA-Z0-9]+") + return re.ReplaceAllLiteralString(str, "-") +}