Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Update auf Contao 4.4: böse Falle bei "Metadaten überschreiben" für Bilder #888

Closed
frontendschlampe opened this issue Jun 22, 2017 · 14 comments
Closed

Update auf Contao 4.4: böse Falle bei "Metadaten überschreiben" für Bilder #888

frontendschlampe opened this issue Jun 22, 2017 · 14 comments
Assignees
@leofeyer
Labels
bug
Milestone
4.4

Comments

@frontendschlampe
Copy link
Contributor

Wir haben ein Update von Contao 4.3.x auf Contao 4.4 gemacht und mit Contao 4.4 wurde ja die Möglichkeit eingeführt, dass bei Bildern nicht sofort die Metadaten angezeigt werden, sondern erst auf Aktivierung einer Checkbox. Contao 4.4 deaktiviert diese Checkbox jedoch im Standard. Das ist bei einem Update natürlich sehr bescheiden, denn alle bereits eingegebenen Metadaten werden nun ignoriert und wir müssen alle Elemente durchgehen und wieder aktivieren.

Es wäre bei einem Update sinnvoller, vorher zu prüfen, ob eines der Metadatenfelder ausgefüllt ist. Wenn ja, dann muss der Punkt "Metadaten überschreiben" aktiviert werden.
@discordier
Copy link
Contributor

Fix in the DB by issuing:

UPDATE tl_content
SET overwriteMeta = 1
WHERE 
(alt <> '') OR 
(imageTitle <> '') OR 
(imageUrl <> '') OR 
(caption <> '')

@asaage
Copy link

asaage commented Jun 22, 2017

contao/core#7774 (comment)

@frontendschlampe
Copy link
Contributor Author

contao/core#7774 (comment)

Das wurde aber nicht gemacht! Das Feld ist einfach deaktiviert.

@fenepedia
Copy link

Ich kann das Problem beim Update bestätigen. Ist mir auch schon aufgefallen.

@leofeyer leofeyer added the bug label Jun 25, 2017
@leofeyer leofeyer added this to the 4.4.1 milestone Jun 25, 2017
@jankout
Copy link

jankout commented Jun 29, 2017

Leider ist das bei mir auch passiert und damit sind alle Metainformationen verschwunden. Weiß jemand, wie ich das von der alten Datenbank in die neue übertragen kann? Danke

@fritzmg
Copy link
Contributor

fritzmg commented Jun 29, 2017

Die Daten sind nicht verschwunden. Sind sind ja immer noch da, nur das Feld tl_content.overwriteMeta ist bei diesen Elementen nicht auf 1 gesetzt. Siehe #888 (comment)

@jankout
Copy link

jankout commented Jun 29, 2017

Der Blick in der Datenbank sagt mir allerdings etwas anderes. Bei mir steht in der Tabelle tl_files in der Spalte meta überall NULL

@fritzmg
Copy link
Contributor

fritzmg commented Jun 29, 2017

Hier in diesem Issue geht es aber um etwas anderes. Du kannst ja Community Forum nach Ratschlägen Fragen, warum deine Meta Daten nicht übernommen worden sind.

@jankout
Copy link

jankout commented Jun 29, 2017 •
edited
Loading

Allerdings passierte das nicht nur bei einer Installation. Daher gehe davon aus, dass es sich auch um einen Fehler handelt. Daher passt es hierher, oder?

Ich habe von Contao 3.5.x auf 4.4 aktualisiert.

@fritzmg
Copy link
Contributor

fritzmg commented Jun 29, 2017 •
edited
Loading

Daher passt es hierher, oder?

Nein, hier geht es um etwas anderes. Hier geht es um den Umstand, dass bei einem Upgrade auf Contao 4.4 (egal von welcher Contao Version) nicht berücksichtigt wird, dass tl_content.overwriteMeta auf true gesetzt werden müsste, wenn bei einem Inhaltselement Meta Daten für das singleSRC eingegeben wurden. Somit werden diese Daten im Frontend auch nicht ausgegeben.

@leofeyer leofeyer self-assigned this Jul 3, 2017
leofeyer added a commit to contao/installation-bundle that referenced this issue Jul 3, 2017
@leofeyer
Correctly set the "overwriteMeta" field during the database update (see
30506c1 
contao/core-bundle#888).
@leofeyer
Copy link
Member

leofeyer commented Jul 3, 2017

Behoben in contao/installation-bundle@30506c1.

@leofeyer leofeyer closed this as completed Jul 3, 2017
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Jul 12, 2017
Update contao44 to 4.4.1.
6cca93a 
### 4.4.1 (2017-07-12)

 * Prevent arbitrary PHP file inclusions in the back end (see CVE-2017-10993).
 * Correctly handle subpalettes in "edit multiple" mode (see #946).
 * Correctly show the DCA picker in the site structure (see #906).
 * Correctly update the style sheets if a format definition is
   enabled/disabled (see #893).
 * Always show the "show from" and "show until" fields (see #908).
 * Correctly set the "overwriteMeta" field during the database update (see
   contao/core-bundle#888).
@Xendiadyon
Copy link

overrideMeta gibt's auch in tl_news, oder? Wird es gerade nur in tl_content gefixt oder auch in tl_news?

@fritzmg
Copy link
Contributor

fritzmg commented Jul 14, 2017

Wurde momentan nur für tl_content gemacht: https://github.com/contao/installation-bundle/blob/4.4.1/src/Database/Version440Update.php#L42

@leofeyer
Copy link
Member

Geändert in contao/installation-bundle@4c096c0.

netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Jul 16, 2017
@bsiegert
Pullup ticket #5514 - requested by taca
5388cb2 
www/contao44: security fix

Revisions pulled up:
- www/contao44/Makefile                                         1.2
- www/contao44/PLIST                                            1.2
- www/contao44/distinfo                                         1.2

---
   Module Name:	pkgsrc
   Committed By:	taca
   Date:		Wed Jul 12 14:52:02 UTC 2017

   Modified Files:
   	pkgsrc/www/contao44: Makefile PLIST distinfo

   Log Message:
   Update contao44 to 4.4.1.

   ### 4.4.1 (2017-07-12)

    * Prevent arbitrary PHP file inclusions in the back end (see CVE-2017-10993).
    * Correctly handle subpalettes in "edit multiple" mode (see #946).
    * Correctly show the DCA picker in the site structure (see #906).
    * Correctly update the style sheets if a format definition is
      enabled/disabled (see #893).
    * Always show the "show from" and "show until" fields (see #908).
    * Correctly set the "overwriteMeta" field during the database update (see
      contao/core-bundle#888).
@leofeyer leofeyer modified the milestones: 4.4.1, 4.4 May 14, 2019
leofeyer added a commit that referenced this issue Nov 4, 2019
@leofeyer
Hide the breadcrumb menu if the node is not within the given path (see …
8dfb402 
…#888)

Description
-----------

Fixes #856

@ausi I am not 100% sure if this is the best place to fix the issue. Can you check as well please?

Commits
-------

c7d12ce5 Hide the breadcrumb menu if the node is not within the given path (see #856)
492b74b0 Also initialize the picker if $_GET['act'] is set
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Jan 14, 2020
@bsiegert
Pullup ticket #5514 - requested by taca
a4a0753 
www/contao44: security fix

Revisions pulled up:
- www/contao44/Makefile                                         1.2
- www/contao44/PLIST                                            1.2
- www/contao44/distinfo                                         1.2

---
   Module Name:	pkgsrc
   Committed By:	taca
   Date:		Wed Jul 12 14:52:02 UTC 2017

   Modified Files:
   	pkgsrc/www/contao44: Makefile PLIST distinfo

   Log Message:
   Update contao44 to 4.4.1.

   ### 4.4.1 (2017-07-12)

    * Prevent arbitrary PHP file inclusions in the back end (see CVE-2017-10993).
    * Correctly handle subpalettes in "edit multiple" mode (see #946).
    * Correctly show the DCA picker in the site structure (see #906).
    * Correctly update the style sheets if a format definition is
      enabled/disabled (see #893).
    * Always show the "show from" and "show until" fields (see #908).
    * Correctly set the "overwriteMeta" field during the database update (see
      contao/core-bundle#888).
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue May 27, 2020
@bsiegert
Pullup ticket #5514 - requested by taca
f923660 
www/contao44: security fix

Revisions pulled up:
- www/contao44/Makefile                                         1.2
- www/contao44/PLIST                                            1.2
- www/contao44/distinfo                                         1.2

---
   Module Name:	pkgsrc
   Committed By:	taca
   Date:		Wed Jul 12 14:52:02 UTC 2017

   Modified Files:
   	pkgsrc/www/contao44: Makefile PLIST distinfo

   Log Message:
   Update contao44 to 4.4.1.

   ### 4.4.1 (2017-07-12)

    * Prevent arbitrary PHP file inclusions in the back end (see CVE-2017-10993).
    * Correctly handle subpalettes in "edit multiple" mode (see #946).
    * Correctly show the DCA picker in the site structure (see #906).
    * Correctly update the style sheets if a format definition is
      enabled/disabled (see #893).
    * Always show the "show from" and "show until" fields (see #908).
    * Correctly set the "overwriteMeta" field during the database update (see
      contao/core-bundle#888).
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Oct 14, 2021
@bsiegert
Pullup ticket #5514 - requested by taca
1abdc54 
www/contao44: security fix

Revisions pulled up:
- www/contao44/Makefile                                         1.2
- www/contao44/PLIST                                            1.2
- www/contao44/distinfo                                         1.2

---
   Module Name:	pkgsrc
   Committed By:	taca
   Date:		Wed Jul 12 14:52:02 UTC 2017

   Modified Files:
   	pkgsrc/www/contao44: Makefile PLIST distinfo

   Log Message:
   Update contao44 to 4.4.1.

   ### 4.4.1 (2017-07-12)

    * Prevent arbitrary PHP file inclusions in the back end (see CVE-2017-10993).
    * Correctly handle subpalettes in "edit multiple" mode (see #946).
    * Correctly show the DCA picker in the site structure (see #906).
    * Correctly update the style sheets if a format definition is
      enabled/disabled (see #893).
    * Always show the "show from" and "show until" fields (see #908).
    * Correctly set the "overwriteMeta" field during the database update (see
      contao/core-bundle#888).
netbsd-srcmastr pushed a commit to NetBSD/pkgsrc that referenced this issue Jan 18, 2023
@bsiegert
Pullup ticket #5514 - requested by taca
4027241 
www/contao44: security fix

Revisions pulled up:
- www/contao44/Makefile                                         1.2
- www/contao44/PLIST                                            1.2
- www/contao44/distinfo                                         1.2

---
   Module Name:	pkgsrc
   Committed By:	taca
   Date:		Wed Jul 12 14:52:02 UTC 2017

   Modified Files:
   	pkgsrc/www/contao44: Makefile PLIST distinfo

   Log Message:
   Update contao44 to 4.4.1.

   ### 4.4.1 (2017-07-12)

    * Prevent arbitrary PHP file inclusions in the back end (see CVE-2017-10993).
    * Correctly handle subpalettes in "edit multiple" mode (see #946).
    * Correctly show the DCA picker in the site structure (see #906).
    * Correctly update the style sheets if a format definition is
      enabled/disabled (see #893).
    * Always show the "show from" and "show until" fields (see #908).
    * Correctly set the "overwriteMeta" field during the database update (see
      contao/core-bundle#888).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@leofeyer leofeyer

Labels
bug
Projects
None yet
Milestone
4.4
Development

No branches or pull requests
8 participants
@fenepedia @discordier @leofeyer @frontendschlampe @Xendiadyon @jankout @asaage @fritzmg