Modify

Opened 6 years ago

Closed 6 years ago

Last modified 6 years ago

#1780 closed Bug/Fehler (fixed)

Admin Button in der Adminbar nach Update auf 2.0.5.0 ohne Link zum Adminbereich

Reported by: FräuleinGarn Owned by: Torsten Riemer
Priority: normal Milestone: modified-shop-2.0.5.1
Component: Template Version: 2.0.5.0
Keywords: Cc:
Blocked By: Blocking:

Description (last modified by Torsten Riemer)

Siehe: Admin Button in der Adminbar fehlt

in der Datei von tpl_modified_responsive_1 ist noch enthalten

// start
if ($admin_access['start'] == '1') {
  $box_smarty->assign('START', xtc_href_link_admin(FILENAME_START,'', 'NONSSL'));
}

statt

// start
$box_smarty->assign('START', xtc_href_link_admin(FILENAME_START,'', 'NONSSL'));

wie es in der Anleitung steht: Tutorial: Template eines xt:Commerce Shops in der modified eCommerce Shopsoftware weiter verwenden - Wegfall der Adminberechtigung für "start"

Anscheinend kommt es dadurch bei einigen zu Fehlern. In meinem Testshop allerdings nicht.

Attachments (0)

Change History (12)

comment:1 by Markus Rehm, 6 years ago

Hi Timm,

da muss ich widersprechen ... dem ist nicht so. Ich hab mir die 2.0.5.0 gerade einmal runtergeladen und reingeschaut ... da ist die if-Abfrage nicht mehr drin. Die wurde in rev. 12346 rausgenommen.

Markus

comment:2 by Torsten Riemer, 6 years ago

Description: modified (diff)
Summary: /templates/tpl_modified_responsive1/source/boxes/admin.php in 2.0.5.0 noch auf Stand 2.0.4.2Admin Button in der Adminbar fehlt nach Update auf 2.0.5.0

comment:3 by Torsten Riemer, 6 years ago

Resolution: fixed
Status: newclosed

Das Problem ist ein anderes: Durch das Datenbankstruktur-Update wurden die Einträge für "start" und "blz_update" wieder hinzugefügt, obwohl diese beim Datenbank-Update bereits gelöscht wurden. An der Stelle wurde vergessen diese Einträge auch aus der "modified.sql" zu entfernen, was wir bereits in r12625 & r12626 nachgeholt haben.
Für Shopversion 2.0.5.1 werden die beiden Einträge zusätzlich r12628 auch nochmal über das Datenbank-Update entfernt.

Der Code der "/source/boxes/admin.php" in den Kauftemplates unterscheidet sich absichtlich zu dem in den mitgelieferten Templates, da wir in den Kauftemplates eine Abwärtskompatibilität gewährleisten müssen!

Um das Problem zu beheben reicht ein erneutes Ausführen über phpMyAdmin von:

#Tomcraft - 2020-03-10 - removed blz update again due to r12625, so the database structure update will not add the field again
ALTER TABLE `admin_access` DROP `blz_update`;

#Tomcraft - 2020-03-10 - removed start again due to r12626, so the database structure update will not add the field again
ALTER TABLE `admin_access` DROP `start`;

comment:4 by FräuleinGarn, 6 years ago

Es geht um das kostenpflichtige tpl_modified_responsive1 und nicht das standardmäßige tpl_modified_responsive.

comment:5 by FräuleinGarn, 6 years ago

@Tomcraft
Dann ist die Anleitung im Wiki falsch, weil dort das tpl_modified_responsive1 als zu ändern aufgelistet ist.

in reply to:  5 ; comment:6 by Torsten Riemer, 6 years ago

Replying to FräuleinGarn:

Es geht um das kostenpflichtige tpl_modified_responsive1 und nicht das standardmäßige tpl_modified_responsive.

Das Problem ist das selbe. Schrieb ich doch bzgl. der Unterschiede zwischen Kauftemplates und mitgelieferter Templates.

Replying to FräuleinGarn:

@Tomcraft
Dann ist die Anleitung im Wiki falsch, weil dort das tpl_modified_responsive1 als zu ändern aufgelistet ist.

Nein! Die ist korrekt für diejenigen, die das Template vor Veröffentlichung der Shopversion 2.0.5.0 gekauft haben und die notwendige Änderung in der "/source/boxes/admin.php" noch nicht haben!

in reply to:  6 comment:7 by FräuleinGarn, 6 years ago

Replying to Tomcraft:

Nein! Die ist korrekt für diejenigen, die das Template vor Veröffentlichung der Shopversion 2.0.5.0 gekauft haben und die notwendige Änderung in der "/source/boxes/admin.php" noch nicht haben!

Versteh ich nicht so ganz. Ich hab das Update doch bei euch nach erscheinen der 2.0.5.0 gekauft, aber die Änderung ist nicht drin.

Ich verstehe, dass jemand der das Template früher gekauft hat und kein Updatepaket bei euch buchen möchte, dass er es selbst anpassen kann auf Grund der Anleitung. Wenn ich aber ein aktuelles Template bei euch kaufe, dann muss die Zeile da schon geändert sein. Woher weiß ich sonst, was noch alles aus der Anleitung fehlt um Abwärtskompatibilität zu gewährleisten? Ich bezahle doch dafür, damit ich mir die Anleitung nicht mehr antun muss.

Oder ich versteh da was falsch.

comment:8 by Torsten Riemer, 6 years ago

Du stehst gerade aber auf dem Schlauch oder ich drücke mich undeutlich aus. ;-)

Dein Template beinhaltet den folgenden Code an der Stelle:

// start
if (!isset($admin_access['start']) || $admin_access['start'] == '1') {
  $box_smarty->assign('START', xtc_href_link_admin(FILENAME_START, '', 'NONSSL'));
}

Und das ist bereits der Code inkl. Fallback für ältere Shopversionen. In der Anleitung brauchen wir aber keinen Fallback, daher der Unterschied der Anleitung zum Code aus den bereits für 2.0.5.0 angepassten Templates. ;-)

Dir fehlen dann nur die Änderungen ab Shopversion 2.0.5.1 und nur die zählen für dich.

comment:9 by Torsten Riemer, 6 years ago

Summary: Admin Button in der Adminbar fehlt nach Update auf 2.0.5.0Admin Button in der Adminbar nach Update auf 2.0.5.0 ohne Link zum Adminbereich

comment:10 by Torsten Riemer, 6 years ago

Owner: set to Torsten Riemer

In 12625:

fix #1780 - Add missing changes to r12335 (update banktransfer)

comment:11 by Torsten Riemer, 6 years ago

In 12626:

fix #1780 - Add missing changes to r12345 (remove start from admin access)

comment:12 by Torsten Riemer, 6 years ago

In 12628:

fix #1780 - Removed blz_update and start from admin_access again due to r12625 & r12626, so the database structure update will not add the fields again

Modify Ticket

Action
as closed The owner will remain Torsten Riemer.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.